Closed Bug 948475 Opened 6 years ago Closed 6 years ago
Element .select() for <input type=number>, contrary to the HTML5 spec
I didn't implement HTMLInputElement.select() for <input type=number> because the spec says it doesn't apply: http://www.whatwg.org/specs/web-apps/current-work/multipage/states-of-the-type-attribute.html#number-state-%28type=number%29 Turns out that Chrome implements it though, and pdf.js relies on it working: https://github.com/mozilla/pdf.js/pull/4020#issuecomment-30243390 I think rather than select() just doing nothing, it seems reasonable to forward the call on to its anonymous text control grandchild.
Err, disregard the viewer.css change - I was testing pdf.js locally. Removed from my version of the patch.
Can you please bring this up on the whatwg list so that we can change the spec? Thanks!
I'm very late starting my PTO. I'll let someone else do that or it can wait until January.
Comment on attachment 8345346 [details] [diff] [review] patch Isn't this missing number-selected.html and number-selected-ref.html
Attachment #8345346 - Flags: review?(bugs) → review-
Oops, sorry. Pushed this by mistake when qfinishing and pushing bug 946184 since this was also applied. I'll leave it in since I'm sure you're fine with this modulo seeing the tests. Let me know if you want any changes. https://hg.mozilla.org/integration/mozilla-inbound/rev/fa744c4a8948
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
Comment on attachment 8345521 [details] [diff] [review] patch Bug caused by (feature/regressing bug #): 344616 and its dependencies User impact if declined: for example, the page number field in pdf.js doesn't select Testing completed (on m-c, etc.): landed m-i, merged to m-c Risk to taking this patch (and alternatives if risky): very low String or IDL/UUID changes made by this patch: none We're early in the cycle so hopefully this is fine to uplift.
Attachment #8345521 - Flags: approval-mozilla-aurora?
Attachment #8345521 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.