Skip to content

Commit 2395c7a

Browse files
authored
Fix slider display update and analytics (#1678)
Fix slider display update and analytics
2 parents b98775d + 65ab462 commit 2395c7a

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

src/js/tabs/pid_tuning.js

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1442,9 +1442,11 @@ TABS.pid_tuning.initialize = function (callback) {
14421442
const NON_EXPERT_SLIDER_MIN = 0.7;
14431443

14441444
$('input[name="expertModeCheckbox"]').change(function() {
1445-
TuningSliders.setExpertMode($(this).is(':checked'));
1446-
TuningSliders.updatePidSlidersDisplay();
1447-
TuningSliders.updateFilterSlidersDisplay();
1445+
if (TuningSliders.expertMode !== $(this).is(':checked')) {
1446+
TuningSliders.setExpertMode($(this).is(':checked'));
1447+
TuningSliders.updatePidSlidersDisplay();
1448+
TuningSliders.updateFilterSlidersDisplay();
1449+
}
14481450
});
14491451

14501452
$('#dMinSwitch').change(function() {
@@ -1600,7 +1602,7 @@ TABS.pid_tuning.initialize = function (callback) {
16001602
self.analyticsChanges['PidTuningSliders'] = "Off";
16011603
});
16021604
// update on filter value or type changes
1603-
$('.pid_filter input, .pid_filter select').on('input', function() {
1605+
$('.pid_filter tr:not(.newFilter) input, .pid_filter tr:not(.newFilter) select').on('input', function() {
16041606
TuningSliders.updateFilterSlidersDisplay();
16051607
if (TuningSliders.filterGyroSliderUnavailable) {
16061608
self.analyticsChanges['GyroFilterTuningSlider'] = "Off";
@@ -1610,7 +1612,7 @@ TABS.pid_tuning.initialize = function (callback) {
16101612
}
16111613
});
16121614
// update on filter switch changes
1613-
$('.inputSwitch input').change(() => $('.pid_filter input').trigger('input'));
1615+
$('.pid_filter tr:not(.newFilter) .inputSwitch input').change(() => $('.pid_filter input').triggerHandler('input'));
16141616

16151617
$('.tuningHelp').hide();
16161618
} else {

0 commit comments

Comments
 (0)