-
Notifications
You must be signed in to change notification settings - Fork 544
Update/add source header comments (continued) #1226
Conversation
sections/introduction.include
Outdated
- some privacy considerations | ||
- an introduction to HTML itself, how to write it, etc. | ||
- non-normative conformance requirements for content | ||
- links to some other specs: charmod, unicode security, and some accessibility specs |
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.
Unicode - is a proper noun so should start with a capital letter. I think.
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.
updated
|
||
Interactive Elements | ||
|
||
This source produces section 4.7: Embedded Content |
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 think this header is incorrect. Appears to be a duplicate of the one in semantics-embedded-content.
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.
thanks!
|
||
It covers | ||
|
||
- media elements: |
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 don't think so, I think it covers details/summary, dialog, and commands
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.
wow. sorry about that. I don't know what happened there.
Just pushed a fix!
PR edit:
|
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.
Looks good. Many thanks
Continuing to work through issue #1201
This commit adds new source header comments for the following files:
This commit touches other previously edited source files to continue to normalize the way we format these source headers.