-
Notifications
You must be signed in to change notification settings - Fork 186
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
css/properties.json is incomplete #149
Comments
Hi! thanks for finding this! Do you think you could submit a PR for these? |
I tried, but I have no idea what to fill in for the syntax and other fields. Also, the file should be checked for other missing properties as well? Not sure how to do that either. |
@teoli2003 What’s the best source of truth for what should be in this repository? https://www.w3.org/TR/css-2017/ ?? I’d love to contribute. Just not sure where to start. |
Hi! Properties that are listed here are the ones that has been/are implemented by one browser that shipped. [We can make some exceptions, but that it the idea] Then for the values of each entry, that is syntax and so, we prime the implementations. If they diverge, we usually follow the spec if it tries to solve the divergence. CSS 2017 is usually a good start, though I'm not sure about the implementation status (in browsers) of things like CSS Speech, but we will definitively incoroporate additional drafts. The MDN link you gave should use this data/properties.json as a source of truth. Hope it helps. |
Thank you @pkuczynski! 🙏 |
#184 is still open. |
You welcome @Kapeli |
@wbamberg shall we close this one? |
Some properties that are missing from css/properties.json:
The text was updated successfully, but these errors were encountered: