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

More fixes for FACE attribute issues #5022

Merged
merged 2 commits into from
Oct 19, 2019
Merged

Conversation

domenic
Copy link
Member

@domenic domenic commented Oct 19, 2019

The first commit here is a normative addition of authoring advice. The second is more mechanical tweaks.

These should probably be merged around the same time as whatwg/html-build#210, ideally right after. I'm happy to also throw in fixes for the dir attribute annotations as discussed in that PR, if we agree on which way to fix it.


/custom-elements.html ( diff )
/indices.html ( diff )

@domenic domenic added topic: custom elements Relates to custom elements (as defined in DOM and HTML) topic: forms labels Oct 19, 2019
@domenic domenic requested a review from annevk October 19, 2019 00:29
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.

Providing no annotation for bdi/bdo or a specific one both seem fine to me.

* Adds an annotation to prevent "is" and "dir" from getting
  over-explained, per whatwg/html-build#210.
* Removes the double description of the disabled attribute
  (attr-fe-disabled), once for form elements and once for
  form-associated custom elements. There is only one <dfn>
  so there should be only one attribute index entry.
@domenic domenic force-pushed the more-face-readonly-corrections branch from 1024799 to 2fe8ef5 Compare October 19, 2019 15:10
@domenic domenic merged commit 3909818 into master Oct 19, 2019
@domenic domenic deleted the more-face-readonly-corrections branch October 19, 2019 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: custom elements Relates to custom elements (as defined in DOM and HTML) topic: forms
Development

Successfully merging this pull request may close these issues.

2 participants