-
Notifications
You must be signed in to change notification settings - Fork 2.9k
Define an optional value and use it for setting dialog returnValue #11280
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
Conversation
Define a new 'optional value' for buttons and submit inputs, this is used instead of 'value' when setting a dialog's returnValue.
@annevk hopefully this is roughly what you had in mind? |
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.
I had forgotten about this, but this looks like an improvement over the status quo.
Add submit button check. ought to be used*
a54c9f8
to
cadc51c
Compare
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.
Modulo Infra-alignment this looks good. Thanks for tackling it!
Thanks for the review, this should be ready to go |
We now assert the actual value of returnValue when the submit button has no attribute. See whatwg/html#11280.
Automatic update from web-platform-tests Update dialog form submission test We now assert the actual value of returnValue when the submit button has no attribute. See whatwg/html#11280. -- wpt-commits: 76c3c693ab36c5a7eb523ec798d60cfd57379423 wpt-pr: 51484 Differential Revision: https://phabricator.services.mozilla.com/D251003
…mmand buttons, Automatic update from web-platform-tests Add more dialog returnValue tests for command buttons See whatwg/html#11280. -- wpt-commits: 975e4f6ae35d7390beab0ba0c871a349f15ea87e wpt-pr: 52278 Differential Revision: https://phabricator.services.mozilla.com/D251005
Automatic update from web-platform-tests Update dialog form submission test We now assert the actual value of returnValue when the submit button has no attribute. See whatwg/html#11280. -- wpt-commits: 76c3c693ab36c5a7eb523ec798d60cfd57379423 wpt-pr: 51484 Differential Revision: https://phabricator.services.mozilla.com/D251003
…mmand buttons, Automatic update from web-platform-tests Add more dialog returnValue tests for command buttons See whatwg/html#11280. -- wpt-commits: 975e4f6ae35d7390beab0ba0c871a349f15ea87e wpt-pr: 52278 Differential Revision: https://phabricator.services.mozilla.com/D251005
Automatic update from web-platform-tests Update dialog form submission test We now assert the actual value of returnValue when the submit button has no attribute. See whatwg/html#11280. -- wpt-commits: 76c3c693ab36c5a7eb523ec798d60cfd57379423 wpt-pr: 51484 Differential Revision: https://phabricator.services.mozilla.com/D251003
…mmand buttons, Automatic update from web-platform-tests Add more dialog returnValue tests for command buttons See whatwg/html#11280. -- wpt-commits: 975e4f6ae35d7390beab0ba0c871a349f15ea87e wpt-pr: 52278 Differential Revision: https://phabricator.services.mozilla.com/D251005
Automatic update from web-platform-tests Update dialog form submission test We now assert the actual value of returnValue when the submit button has no attribute. See whatwg/html#11280. -- wpt-commits: 76c3c693ab36c5a7eb523ec798d60cfd57379423 wpt-pr: 51484 Differential Revision: https://phabricator.services.mozilla.com/D251003 UltraBlame original commit: 984064be0606698b570273a83a79d2fda0276504
…mmand buttons, Automatic update from web-platform-tests Add more dialog returnValue tests for command buttons See whatwg/html#11280. -- wpt-commits: 975e4f6ae35d7390beab0ba0c871a349f15ea87e wpt-pr: 52278 Differential Revision: https://phabricator.services.mozilla.com/D251005 UltraBlame original commit: 4df6b216f6a259ee1077dafe848ddcfde0d8c807
Automatic update from web-platform-tests Update dialog form submission test We now assert the actual value of returnValue when the submit button has no attribute. See whatwg/html#11280. -- wpt-commits: 76c3c693ab36c5a7eb523ec798d60cfd57379423 wpt-pr: 51484 Differential Revision: https://phabricator.services.mozilla.com/D251003 UltraBlame original commit: 984064be0606698b570273a83a79d2fda0276504
…mmand buttons, Automatic update from web-platform-tests Add more dialog returnValue tests for command buttons See whatwg/html#11280. -- wpt-commits: 975e4f6ae35d7390beab0ba0c871a349f15ea87e wpt-pr: 52278 Differential Revision: https://phabricator.services.mozilla.com/D251005 UltraBlame original commit: 4df6b216f6a259ee1077dafe848ddcfde0d8c807
Automatic update from web-platform-tests Update dialog form submission test We now assert the actual value of returnValue when the submit button has no attribute. See whatwg/html#11280. -- wpt-commits: 76c3c693ab36c5a7eb523ec798d60cfd57379423 wpt-pr: 51484 Differential Revision: https://phabricator.services.mozilla.com/D251003 UltraBlame original commit: 984064be0606698b570273a83a79d2fda0276504
…mmand buttons, Automatic update from web-platform-tests Add more dialog returnValue tests for command buttons See whatwg/html#11280. -- wpt-commits: 975e4f6ae35d7390beab0ba0c871a349f15ea87e wpt-pr: 52278 Differential Revision: https://phabricator.services.mozilla.com/D251005 UltraBlame original commit: 4df6b216f6a259ee1077dafe848ddcfde0d8c807
Define a new 'optional value' for buttons and submit inputs, this is used instead of 'value' when setting a dialog's returnValue.
Fixes #11092
(See WHATWG Working Mode: Changes for more details.)
/form-control-infrastructure.html ( diff )
/form-elements.html ( diff )
/input.html ( diff )
/interactive-elements.html ( diff )