IonDen / ion.rangeSlider

jQuery only range slider
http://ionden.com/a/plugins/ion.rangeSlider/en.html
MIT License
2.55k stars 508 forks source link

ion.rangeSlider in ion.tabs #101

Closed wsnnn closed 10 years ago

wsnnn commented 10 years ago

Found that if ion.rangeSlider is in ion.tabs, that slider is not initialized, if this particular tab no class ionTabs__item_state_active. It turns out that if you have multiple tabs and they all contain a slider, then all must be active. When the page loads, this leads to flicker tabs, but the sliders are working correctly. Killed half a day to figure it out.

Обнаружил, что если ion.rangeSlider находится в ion.tabs, то slider не инициализируется, если у данного конкретного таба нет класса ionTabs__item_state_active. Выходит, что если у вас несколько табов и они все содержат slider, то все должны быть активными. При загрузке страницы это приводит к морганию табов, но зато слайдеры работают правильно. Убил пол дня, чтобы это вычислить.

IonDen commented 10 years ago

Привет, не совсем так. Внутри табов слайдер разумеется не запускается корректно, для этого у табов есть событие onChange. Которое вызывается когда вы меняете активный таб. В обработчике этого события достаточно вызвать метод update у слайдера и все будет нормально.

wsnnn commented 10 years ago

Да, спасибо, так и сделал. Просто про это сочетание может в хелпах где упомянуть, а то не все ведь умные, много таких как я:)