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

Addition: update and add missing attributes from HTML #469

Merged
merged 2 commits into from
Apr 27, 2023

Conversation

scottaohara
Copy link
Member

@scottaohara scottaohara commented Apr 11, 2023

related to #400

none of the updated or added attributes require implementation updates as updates either provide clarifications that reflect current implementations, or the attributes are not mapped.

creates separate table mappings for:

  • bdo[dir]
  • fieldset[disabled]

adds:

  • link[disabled]
  • [enterkeyhint]
  • img, link, script[fetchpriority]

Preview | Diff

@scottaohara scottaohara changed the title add missing attributes from HTML Addition: update and add missing attributes from HTML Apr 11, 2023
@spectranaut spectranaut self-requested a review April 13, 2023 17:33
@spectranaut
Copy link
Contributor

@douggeoffray to review

Copy link
Contributor

@spectranaut spectranaut left a comment

Choose a reason for hiding this comment

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

all looks ok to me!

@ariellalgilmore ariellalgilmore removed their assignment Apr 17, 2023
@ariellalgilmore ariellalgilmore self-requested a review April 17, 2023 16:37
Copy link

@ariellalgilmore ariellalgilmore left a comment

Choose a reason for hiding this comment

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

LGTM!

@scottaohara scottaohara merged commit 04753c9 into gh-pages Apr 27, 2023
1 check passed
@scottaohara scottaohara deleted the attr-updates branch April 27, 2023 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants