bendavis78 / paper-time-picker

Material design time picker component for polymer
http://bendavis78.github.io/paper-time-picker/
MIT License
89 stars 61 forks source link

Second open() skips hours #45

Closed rwestlund closed 8 years ago

rwestlund commented 8 years ago

Versions

Regardless of how many times the picker is opened, hours should be shown first.

Actual behavior

When I open the time picker a second time, it shows the minute picker, bypassing the hour picker.

Cause

It looks like this.mode is set to hours in ready(), but nothing else resets it back to hours. It could be reset during open(), or perhaps a reset() function could be added.

Thanks for such a great element :)

bendavis78 commented 8 years ago

Open/close isn't really a feature of the time picker -- that's the job of the dialog. You can set the view property to "hours" when the dialog closes to reset it. I've updated the demo to show how this is done.

rwestlund commented 8 years ago

That works, thanks.