-
Notifications
You must be signed in to change notification settings - Fork 64
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Breaks Chrome input type=date #3
Comments
Using step="any", as is allowed in the specification (http://www.w3.org/TR/html-markup/input.date.html) works fine. |
If I manually set step="any" in the raw HTML, it is not respected, and is replaced with step="undefined". However step="1" is respected (only if value is empty), and provides a partial workaround for this bug. |
I will take a look at this issue. Do you have an example that I can look at? Just want to make sure that I'm looking at the same thing as you are. Thanks Z. |
Right now, my code is applied to the backend of a cms, so it is not publicly available. html5Forms.js is called in the head like this:
<script src=https://github.com/zoltan-dulac/html5Forms.js/issues/"path/to/html5forms/shared/js/html5Forms.js" data-webforms2-support="all" data-webforms2-force-js-validation="true"></script>">
The HTML that generates the issue is:
Once the javascript is run, it becomes:
Chrome chokes on the However, if I edit the raw HTML to include
This is ignored by the javascript and becomes:
Alternatively, if I add
The javascript respects this, and all works as expected:
But if the value parameter is assigned:
The javascript, once again, ignores the
Generally, I think it is bad form to add Other than this, the code is fantastic. I greatly appreciate all the hard work you put in to this. You have made the web a better place! |
If the above is insufficient to recreate the issue, email me at jeff at kingdesk dot com, and I will get you access to the WordPress backend where this is being applied. |
I have prepared test case that demonstrates the bug(s) here: http://test.kingdesk.com/date-input/ |
FYI the test case page (http://test.kingdesk.com/date-input/) works fine in current chrome on OS X. |
@kingjeffrey , @erutan : Thanks for your work looking into this. Yeah -- it seems like Google fixed this in Chrome already, so I guess we can close this. I'll take a look at #13 as well and if it's the same, I'll close that as well. |
The html5Forms.js addition of
step="undefined"
to input type=date causes all dates in Chrome's native drop down (Version 20.0.1132.47) to be unselectable.The text was updated successfully, but these errors were encountered: