This repository was archived by the owner on Dec 15, 2022. It is now read-only.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Requirements
Description of the Change
VSCode will be switching back to using this version of the CSS grammar shortly. However, there are a few minor discrepancies between the version they were using and this one. This is one of two PRs that takes the relevant changes from that grammar and applies it to Atom's. In this PR, I've adjusted the list of properties. For the reviewer's convenience, I've included a property-by-property diff at the end of this description. All of the props added were either on the list of MDN's CSS Reference or their list of SVG Presentation Attributes. SVG Presentation Attributes can be set in CSS.
Alternate Designs
N/A
Benefits
A more up-to-date list of CSS properties
Possible Drawbacks
None foreseeable
Applicable Issues
microsoft/vscode#115480
A diff of the property names: