-
Notifications
You must be signed in to change notification settings - Fork 32
Description
Hi @alanclarke, thank you for putting out this library - seems very useful.
As I began implementing, I noticed a few things which I wanted to bring up -
-
Support for placeholder: I feel like if you don't have a value to initialize with, showing a placeholder would be good user experience. The placeholder could be time or a string (like 'enter time').
-
Guides: This is especially useful when one begins to type, as it guides the user. So, lets say a user starts entering the number I am thinking of being able to show the separator and no text (or grayed out text). This, in conjunction with the a placeholder time makes for a good user experience as the placeholder doesn't disapper but just grays out acting as a guide.
-
When we enter the two digits for the hours, I feel like the cursor should move past the separator seeing how it moved to the next digit previously (when you enter the first hour digit).
-
I noticed that the
inputhastype=text. I wonder if it would be more semantic to have it betype=time?
Thank you!