Skip to content
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

Use normalize-newlines algorithm for setCustomValidity #10350

Merged
merged 2 commits into from
May 17, 2024

Conversation

sefeng211
Copy link
Contributor

@sefeng211 sefeng211 commented May 15, 2024

This fixes #10339

(See WHATWG Working Mode: Changes for more details.)


/form-control-infrastructure.html ( diff )

Copy link
Member

@annevk annevk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, one tiny nit. Are you planning on adding test coverage for this?

source Outdated Show resolved Hide resolved
@sefeng211 sefeng211 force-pushed the normalize-newline-custom-validity branch from 2704544 to 3240163 Compare May 15, 2024 18:28
@sefeng211
Copy link
Contributor Author

Yeah I plan to write a test case for this

@sefeng211
Copy link
Contributor Author

@annevk Anne I've added the test as part of my Gecko changes. Do you think we can merge the PR? I am not sure a MDN issue is needed, I guess not?

@sefeng211 sefeng211 requested a review from annevk May 16, 2024 17:26
@annevk annevk merged commit 904aba1 into whatwg:main May 17, 2024
2 checks passed
moz-wptsync-bot pushed a commit to web-platform-tests/wpt that referenced this pull request May 17, 2024
This is for whatwg/html#10350

Differential Revision: https://phabricator.services.mozilla.com/D210688

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1896923
gecko-commit: c9931f28fd168fb6f179eecf28a87a1a738fde0d
gecko-reviewers: emilio
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this pull request May 17, 2024
moz-wptsync-bot pushed a commit to web-platform-tests/wpt that referenced this pull request May 18, 2024
This is for whatwg/html#10350

Differential Revision: https://phabricator.services.mozilla.com/D210688

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1896923
gecko-commit: c9931f28fd168fb6f179eecf28a87a1a738fde0d
gecko-reviewers: emilio
jamienicol pushed a commit to jamienicol/gecko that referenced this pull request May 20, 2024
i3roly pushed a commit to i3roly/firefox-dynasty that referenced this pull request May 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Should custom validity error message treat \r as newline?
2 participants