-
Notifications
You must be signed in to change notification settings - Fork 49
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
Use <pre class=idl> for IDL blocks #48
Comments
Yeah, someone filed #47 about the spec not being up-to-date with what ReSpec wants. |
@foolip, let me know of others. I'm trying to fix all of those too. |
Closing as duplicate. |
@marcoscaceres, these are the issues for the 7 other specs linked from Blink's IDL that still uses the old style:
We'd love to see these fixed. |
@foolip, thanks so much! I'm on it. Will try my best to do them by end of next week (except web audio, which hopefully they will turn to bikeshed). |
We will be very grateful, I feared we'd have to go fix these ourselves after waiting a few months :) |
I have write on quite a few of those repos ;) |
A friendly request. In https://github.com/mdittmer/web-apis @mdittmer is working on tooling for extracting the IDL from specs linked to from Chromium's source code, for the purpose of comparing the IDLs. This will hopefully be a powerful tool for avoiding accidental differences.
In order to avoid having to use a real browser to scrape the IDL, it's useful if a
<pre class=idl>
block is used. This is one spec that Mark found relying on ReSpec's<dl class=pre>
feature.Note that this is already a ReSpec warning: "Defining WebIDL in
dl
elements is deprecated. Please use Contiguous IDL instead: https://github.com/w3c/respec/wiki/User's-Guide#webidl-support"The text was updated successfully, but these errors were encountered: