Savestate and Resetfilter

[sloothword]

The Savestate is only written when the filter (select, slider etc.) are directly changed.
When using the reset buttons the state change is not registered.

Steps to reproduce:

• Go to http://yadcf-showcase.appspot.com/DOM_source_chosen.html
• Change Slider in Numbers Column
• Reload Page -> State is saved and Slider shows value after change
• Press the filter rest button, reload (without touching the slider)
• The slider still has the smaller range set earlier.

I guess the underlying problem is that the savestate update is only done in rangeNumberSliderChange, which does not get called when resetting the value via button.

Edit:
When commenting out the check for originalEvent in the change listener of the slider, the reset is registered and saved correctly:
Line 1100-1102:

//if (event.originalEvent) {
    rangeNumberSliderChange(table_selector_jq_friendly, event, ui);
//}

But i have no idea why the check was introduced in the first place.

[vedmack]

Thanx for the detailed bug report and pinpointing the bug itself. Untill I will fix it you can use the 0.8.3

[vedmack]

fixed in 0.8.5.beta.1 grab from https://github.com/vedmack/yadcf/tree/master/lab , found other state saving issue (not related to Resetfilter , will fix it in next beta release)