-
Notifications
You must be signed in to change notification settings - Fork 266
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
Merge pr3381 and pr3738 #3757
Merge pr3381 and pr3738 #3757
Conversation
* add support for syntax highlighting * update one example
* added support for syntax highlighting * updated resources link
* syntax highlighting * replaced HTML 4.01 link to living standard
* support for syntax highlighting * replaced HTML 4.01 link to living standard
* syntax highlighting * replaced out of date resource link
Note: this technique should be retired. See: [issue 1234](#1234)
* add support for syntax highlighting * update links from W3C HTML spec to WHATWG * add lang support for change of languages * removed references to XHTML and HTML5
* add support for syntax highlighting * remove references to XHTML * update Resources links * remove test to check that `td` elements have `scope` attribute as that’s invalid HTML now * swapped a male name for a female one
* add support for syntax highlighting * remove references to HTML 4.01
Don’t need that if we’ve removed the change log list
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.
Admittedly didn't check out every single change, but on an initial high-level skim, this all looks kosher
@fstrr merge conflicts resolved ... hope i didn't slip up while cherry-picking the bits (hoping we can merge this quickly before it goes out of sync this way again) |
@patrickhlauke Thanks :) I had a quick skim, and it looks okay. @mbgower @alastc Can we get this prioritized to get reviewed please? It's up to 196 pages, which is going to make it susceptible to more merge conflicts the longer it hangs around. |
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 spot checked a number of files. All had correct changes.
Discussed on TF call 4/12 and Ready for approval. |
Merges #3381 and #3738 to:
Closes #3381
Closes #3738