search

w3c / webidl2.js

WebIDL parser
https://w3c.github.io/webidl2.js/checker/
MIT License
251 stars 63 forks source link

[Bug] w3c webref repo contains "implements" in some files. #795

Closed cyraid closed 1 month ago

cyraid commented 2 months ago

I read on one pull request that implements was removed? In the repo:

https://github.com/w3c/webref/blob/429ee9f4891ee37e467f3d8e8fbe1ab193cabd2e/tr/idl/cssom-view.idl#L175

Text implements GeometryUtils; // like Range

There are 4 statements that use implements, and WebIDL2 parser just won't have it. Should there be a filing on that repo? Or is there something we can do for this one?

For now, what would be the recommended approach or workaround?

saschanaz commented 2 months ago

You probably dont' want to consume /tr/idl as they are old. Try using ed/idl instead.

cyraid commented 2 months ago

But ed/idl for example, has things like CSSMarginRule.CSSMarginDescriptors (in ed/idl/cssom.idl), which CSSMarginDescriptors doesn't exist as for as a search goes.

saschanaz commented 2 months ago

Oh, good catch. CSSMarginDescriptors is being used in https://drafts.csswg.org/cssom/ without being defined, that's why it doesn't exist in ed/idl. Can you file an issue in https://github.com/w3c/csswg-drafts/issues ?

dontcallmedom commented 2 months ago

there is already an issue for this FWIW https://github.com/w3c/csswg-drafts/issues/10106

(we patch the CSSOM IDL in webref for that very reason)

saschanaz commented 1 month ago

As this is not a library bug, closing.