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

chore: use new school WebIDL #231

Merged
merged 2 commits into from
Dec 14, 2016
Merged

chore: use new school WebIDL #231

merged 2 commits into from
Dec 14, 2016

Conversation

marcoscaceres
Copy link
Member

No description provided.

@marcoscaceres
Copy link
Member Author

@beverloo, this is 95% done, but I might need you to do the last bits. In particular, you need to split dictionary and interfaces into their own sections (and then just follow what I did). The reason is that there are things that are defined in both interfaces and dictionaries (e.g., "applicationserverkey"). You can explicitly differentiate and reference between the two by either:

The <a>WhateverDictionary.applicationserverkey</a> does x. 

Or:

The <a data-lt="WhateverDictionary.applicationserverkey">applicationserverkey</a> does x. 

See also how I've added <section data-dfn-for="Thing"> and how <dfn>s are defined. It should be quite straight forward.

@@ -205,7 +205,7 @@
<section id="conformance">
<p>
This specification defines conformance criteria that apply to a single product: the
<dfn data-lt="user agent|user agents">user agent</dfn> that implements the interfaces that
<dfn data-lt="agents|user agents">user agent</dfn> that implements the interfaces that
Copy link
Member Author

Choose a reason for hiding this comment

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

note for data-lt's, you don't need to repeat the defined term - so I removed those.

@martinthomson martinthomson merged commit dad5969 into gh-pages Dec 14, 2016
@martinthomson martinthomson deleted the modernize_idl branch December 14, 2016 09:46
@martinthomson martinthomson mentioned this pull request Dec 14, 2016
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