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

Fix cross-spec links and references #247

Merged
merged 1 commit into from
Jan 28, 2020
Merged

Conversation

tidoust
Copy link
Member

@tidoust tidoust commented Jan 14, 2020

A few editorial updates to fix a few broken links and references:

  • Drop custom dfns that are no longer needed (fixes a few outdated fragments and allows to see which additional dfns the spec needs on top of those that the referenced specs export)
  • Replace "triggered by user activation" with new user activation model (see [Meta issue] Fix external references to HTML user activation behavior whatwg/html#5129). This may warrant another iteration as I'm not clear whether we're supposed to call the "activation notification" steps
  • Update definition of methods to fix IDL links
  • Update reference to Web App Manifest and flag the paragraph as informative
  • Fix links to "action"

Preview | Diff

A few editorial updates to fix a few broken links and references:
- Drop custom dfns that are no longer needed (fixes a few outdated fragments
and allows to see which additional dfns the spec needs on top of those that the
referenced specs export)
- Replace "triggered by user activation" with new user activation model
(see whatwg/html#5129). This may warrant another
iteration as I'm not clear whether we're supposed to call the "activation
notification" steps
- Update definition of methods to fix IDL links
- Update reference to Web App Manifest and flag the paragraph as informative
- Fix links to "action"
@tidoust tidoust requested a review from mounirlamouri January 14, 2020 18:25
@mounirlamouri
Copy link
Member

This LGTM, thanks!

@mounirlamouri mounirlamouri merged commit c701fea into w3c:master Jan 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants