-
Notifications
You must be signed in to change notification settings - Fork 79
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
Include links in idlparsed, css, etc. #642
Comments
@gsnedders, are you suggesting something like the following, where the ID is stored in each property’s object? Or did you mean the whole URL?
|
Probably the whole URL, because we probably don't want everyone having to implement the logic of "how to construct a URL for this" (especially in the face of multipage specs). |
Requested in w3c/webref#642 It was possible to link entries in CSS extracts back to entries in the dfns extract with a bit of logic but: 1. This requires having both CSS extracts and dfns extracts at hand, and the dfns extracts typically aren't included in the `@webref/idl` package. 2. This is somewhat error-prone. This updates the CSS definitions extraction logic to also extract the absolute URL (with fragment) of the definition whenever possible. A slight change of approach was needed as previous extraction logic handled "multi-definitions" (e.g., a table definition that defines multiple properties at once) through a text-based approach, and the code now also needs to track the IDs. Tests already contained IDs, so this update merely refreshes the expected results.
Requested in w3c/webref#642 It was possible to link entries in CSS extracts back to entries in the dfns extract with a bit of logic but: 1. This requires having both CSS extracts and dfns extracts at hand, and the dfns extracts typically aren't included in the `@webref/idl` package. 2. This is somewhat error-prone. This updates the CSS definitions extraction logic to also extract the absolute URL (with fragment) of the definition whenever possible. A slight change of approach was needed as previous extraction logic handled "multi-definitions" (e.g., a table definition that defines multiple properties at once) through a text-based approach, and the code now also needs to track the IDs. Tests already contained IDs, so this update merely refreshes the expected results.
Thanks for the nudge, that proved useful to push the issue to the top of the pile ;) CSS extracts in Webref now include an I'll look into doing the same thing for idlparsed extracts. |
All extracts that didn't have more specific links back to the spec now have such links in For idlparsed extracts, some entries don't have links, usually because the spec does not have the appropriate definitions, see w3c/reffy#1493 for details. Closing the issue as addressed. |
Currently you can derive links by processing the data in
dfns
, however this isn't possible if you're trying to only use npm-released data (especially in the css case—I'm aware there's no release of idlparsed).For example, for a property, it would be nice if the URL to the definition was included within the data in
css
.The text was updated successfully, but these errors were encountered: