bevacqua / rome

:calendar: Customizable date (and time) picker. Opt-in UI, no jQuery!
https://bevacqua.github.io/rome
MIT License
2.91k stars 223 forks source link

Toggle visibility with classes rather than style #132

Closed angusfretwell closed 2 years ago

angusfretwell commented 9 years ago

Finally got to this as promised in #99.

I refactored the methods which toggle the visibility of the date picker container, as well as the time list to apply a class rather than directly modifying the display style. As mentioned in the issue, among other things, this means that rome can be easily animated using CSS animations.

As an aside, apologies in advance if there's anything wrong with my PRs, I don't often contribute to projects other than my own, so I'm not 100% sure of the process I should follow :smile: