Add support for <form rel>
Categories
(Core :: DOM: Forms, enhancement, P2)
Tracking
()
People
(Reporter: ehsan.akhgari, Assigned: avandolder)
References
(Blocks 2 open bugs)
Details
(4 keywords)
Attachments
(1 file)
In a similar fashion to bug 1503681, we would like to experiment with breaking opener references for pages opened via form navigations. On the related dev-platform thread, I said this about the spec side of things: ...it's impossible to block opener reference creation upon form submission right now as far as I can tell. This is actually a bug in the spec. <https://html.spec.whatwg.org/multipage/form-control-infrastructure.html#form-submission-algorithm:the-rules-for-choosing-a-browsing-context-given-a-browsing-context-name> calls into "rules for choosing a browsing context" passing only two arguments, omitting the third one (noopener) <https://html.spec.whatwg.org/multipage/browsers.html#the-rules-for-choosing-a-browsing-context-given-a-browsing-context-name>. The web compat risk here is unknown so it may make sense to land a pref that we can turn on first on Nightly and take things from there.
Reporter | ||
Updated•5 years ago
|
Comment 1•5 years ago
|
||
Might require some docs updates somewhere along the line.
Updated•5 years ago
|
Comment 2•5 years ago
|
||
https://github.com/whatwg/html/pull/4332 details changes to the HTML Standard for this. Note that it also adds the rel content/IDL attribute and the relList IDL attribute, as well as support for noopener, noreferrer, and opener, for full parity with <a> and <area>.
Reporter | ||
Updated•5 years ago
|
Reporter | ||
Updated•5 years ago
|
Updated•5 years ago
|
Updated•5 years ago
|
Comment 3•4 years ago
|
||
Forgot to mention, I added tests in https://github.com/web-platform-tests/wpt/pull/15356.
Updated•3 years ago
|
Updated•2 years ago
|
![]() |
||
Updated•2 years ago
|
Updated•10 months ago
|
Updated•9 months ago
|
Assignee | ||
Updated•9 months ago
|
Assignee | ||
Comment 5•9 months ago
|
||
Assignee | ||
Comment 6•9 months ago
|
||
Pushed by avandolder@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/5ad22146a015 Add support for <form rel>. r=smaug
Comment 8•9 months ago
|
||
Backed out for causing for causing failures in docshell/base/nsDocShell.cpp
Backout link: https://hg.mozilla.org/integration/autoland/rev/8b993ae70da999e10be45c74c940afb8185345e9
Pushed by avandolder@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/704407d7ca31 Add support for <form rel>. r=smaug
Comment 10•9 months ago
|
||
Backed out for causing wpt failures at put-forwards.html.
Backout link: https://hg.mozilla.org/integration/autoland/rev/5a8c175045f06829a94462823ac5da6a791c0f31
Failure log: https://treeherder.mozilla.org/logviewer?job_id=402664304&repo=autoland&lineNumber=40204
Assignee | ||
Comment 11•9 months ago
|
||
Ah, I missed that test. Hopefully I have them all now.
Comment 12•9 months ago
|
||
Pushed by avandolder@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/dac66f1d8342 Add support for <form rel>. r=smaug
Comment 13•8 months ago
|
||
bugherder |
Assignee | ||
Comment 14•8 months ago
|
||
Release Note Request (optional, but appreciated)
[Why is this notable]: Allows use of previously unsupported standard attribute.
[Affects Firefox for Android]: n/a
[Suggested wording]:
[Links (documentation, blog post, etc)]:
Comment 15•8 months ago
|
||
Please provide some suggested wording you would like for this note.
I'll add it to the 111 nightly notes and it will follow through to 111 beta and 111 release.
Assignee | ||
Comment 16•8 months ago
|
||
[Suggested wording]: Use of the rel
attribute is now supported on form
elements, allowing the specification of the relationship between the current document and the form target in a simpler, cross-browser way.
Comment 17•8 months ago
|
||
Thanks, added to Nightly release notes.
Keeping the relnote? flag open to keep it on the radar for inclusion in our final release notes.
Updated•7 months ago
|
Description
•