search

stefangabos / Zebra_Datepicker

A super-lightweight, highly configurable, cross-browser date time picker jQuery plugin
https://stefangabos.github.io/Zebra_Datepicker/
Other
399 stars 183 forks source link

Fixed Firefox bug that broke the timepicker. #145

Closed sebipopa closed 5 years ago

sebipopa commented 5 years ago

The time control element is removed from the DOM and in Firefox the element does not have ".Zebra_Datepicker" as a parent, because it's no longer part of the DOM. By adding this extra condition, everything still works without any masive refactoring. In Chrome the element is removed from the DOM after the condition is verified, that's why this works in Chrome.

An alternative would be to rethink the code so that the DOM is not rewritten every time the date/time changes.

stefangabos commented 5 years ago

I applied a different fix for this one - see 9fa97b5 Thank you!

sebipopa commented 5 years ago

Looks like the auto incrementing functionality no longer works. See https://stefangabos.github.io/Zebra_Datepicker/#time