-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Test input element directionality is only submitted for text and search type #39664
Conversation
…ch type This is in reference to the html issue [9206](whatwg/html#9206)
See pull request for WHATWG HTML 9207 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add coverage for textarea
too?
Stretch goal: might also want to add coverage for new FormData(form)
.
Stretch goal 2: factor out the shared bits of scripts in this directory into a resources/dirname.js
or some such and move dirname-ltr-iframe.html
into that directory too.
Made the changes and simplified the shared parts a bit, I hope the commit is not too large. In the future, we could also test the textarea element in all other dirname test files. My suggestion is that I'll submit those changes when I get to implementing dirname on textarea in firefox. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great, happy to land this with some minor nits addressed.
html/semantics/forms/attributes-common-to-form-controls/resources/dirname.js
Outdated
Show resolved
Hide resolved
html/semantics/forms/attributes-common-to-form-controls/resources/dirname.js
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for cleaning this up and sorry for the delay.
This is in reference to html issue 9206
The standard currently does not enforce that for input elements, the directionality only applies for text and search types. I currently work on changing this and this pull request provides a test for the coming change.