puranjayjain / md-date-time-picker

An implementation of Material Design Picker components in vanilla CSS, JS, and HTML
https://puranjayjain.github.io/md-date-time-picker
Other
275 stars 77 forks source link

Date picker has subtitle active when re-opened #239

Open frobinsonj opened 6 years ago

frobinsonj commented 6 years ago

This causes the incorrect section to be highlighted (see images below). This is a result of these two lines which are run on the dialog being hidden. I fixed it myself by swapping round the actions. Not too sure about creating pull requests etc. so I thought it was best to put this here. Thanks :) https://github.com/puranjayjain/md-date-time-picker/blob/29f967dd8aaf97bdf2f054742a219f434960237a/src/js/mdDateTimePicker.js#L277 https://github.com/puranjayjain/md-date-time-picker/blob/29f967dd8aaf97bdf2f054742a219f434960237a/src/js/mdDateTimePicker.js#L278

Image 1 of issue Image 2 of issue