Skip to content

Remove <label form> and redefine label.form IDL attribute#1120

Merged
zcorpan merged 4 commits into
masterfrom
remove-label-form
Apr 28, 2016
Merged

Remove <label form> and redefine label.form IDL attribute#1120
zcorpan merged 4 commits into
masterfrom
remove-label-form

Conversation

@zcorpan

@zcorpan zcorpan commented Apr 25, 2016

Copy link
Copy Markdown
Member

This implements #787 (comment)

Separated into several commits for easier reviewing.

Comment thread source

</dd>

</dl>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should this perhaps be higher up? Usually the domintro is before we describe any IDL attributes.

@annevk

annevk commented Apr 25, 2016

Copy link
Copy Markdown
Member

LGTM, should we wait for two implementer commitments?

@zcorpan

zcorpan commented Apr 28, 2016

Copy link
Copy Markdown
Member Author

Seems like we can go ahead with this. Need tests though, web-platform-tests/wpt#2924

@zcorpan zcorpan merged commit 99f0f1a into master Apr 28, 2016
@zcorpan zcorpan deleted the remove-label-form branch April 28, 2016 13:49
zcorpan added a commit to web-platform-tests/wpt that referenced this pull request Apr 29, 2016
zcorpan added a commit to web-platform-tests/wpt that referenced this pull request May 13, 2016
arronei pushed a commit to arronei/web-platform-tests that referenced this pull request Jun 14, 2016
ivanzr pushed a commit to ivanzr/web-platform-tests that referenced this pull request Jun 29, 2016
cscott added a commit to fgnass/domino that referenced this pull request Oct 14, 2016
This fixes a test case which was broken by
c8fbfff.

The upstream removal of the form attribute for <label> was
whatwg/html#787, pulled in
whatwg/html#1120, and implemented in
browsers as whatwg/html#787 (comment).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

normative change removal/deprecation Removing or deprecating a feature

Development

Successfully merging this pull request may close these issues.

3 participants