Fix "Response Length" slider showing wrong value (#3659) - "Mad Lab Mode" was falsely "resetting" the slider control, for no obvious reason besides weird implementation.

bdbe04325983e7741f643339af5733926ec0ad99

Cohee <18619528+Cohee1207@users.noreply.github.com>

Signed

public/scripts/power-user.js+10 -8 Viewed

 var originalSliderValues = [];
-async function switchLabMode() {
+async function switchLabMode({ noReset = false } = {}) {
     /*     if (power_user.enableZenSliders && power_user.enableLabMode) {
             toastr.warning("Can't start Lab Mode while Zen Sliders are active")

         $('#labModeWarning').removeClass('displayNone');
         //$("#advanced-ai-config-block input[type='range']").hide()
+        $('#amount_gen_counter').attr('min', '1')
+            .attr('max', '99999')
+            .attr('step', '1');
         $('#amount_gen').attr('min', '1')
             .attr('max', '99999')
             .attr('step', '1');
-    } else {
+    } else if (!noReset) {
         //re apply the original sliders values to each input
         originalSliderValues.forEach(function (slider) {
             $('#' + slider.id)

@@ -539,9 +542,8 @@ async function switchLabMode() {
539	$('#advanced-ai-config-block input[type=\'range\']').show();	542	$('#advanced-ai-config-block input[type=\'range\']').show();
540	$('#labModeWarning').addClass('displayNone');	543	$('#labModeWarning').addClass('displayNone');
541		544
542	$('#amount_gen').attr('min', '16')	545	// To set the correct amount_gen back, we just call the function calculating it correctly
543	.attr('max', '2048')	546	switchMaxContextSize();
544	.attr('step', '1');
545	}	547	}
546	}	548	}
547		549

     $('#prefer_character_prompt').prop('checked', power_user.prefer_character_prompt);
     $('#prefer_character_jailbreak').prop('checked', power_user.prefer_character_jailbreak);
     $('#enableZenSliders').prop('checked', power_user.enableZenSliders).trigger('input');
-    $('#enableLabMode').prop('checked', power_user.enableLabMode).trigger('input');
+    $('#enableLabMode').prop('checked', power_user.enableLabMode).trigger('input', { fromInit: true });
     $(`input[name="avatar_style"][value="${power_user.avatar_style}"]`).prop('checked', true);
     $(`#chat_display option[value=${power_user.chat_display}]`).attr('selected', true).trigger('change');
     $('#chat_width_slider').val(power_user.chat_width);

         saveSettingsDebounced();
     });
-    $('#enableLabMode').on('input', function () {
+    $('#enableLabMode').on('input', function (event, { fromInit = false } = {}) {
         const value = !!$(this).prop('checked');
         if (power_user.enableZenSliders === true && value === true) {
             //disallow Lab Mode if ZenSliders are active

+        }
         power_user.enableLabMode = value;
-        switchLabMode();
+        switchLabMode({ noReset: fromInit });
         saveSettingsDebounced();
     });