-
Notifications
You must be signed in to change notification settings - Fork 29.1k
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
Consider adopting a new CSS syntax highlighting grammar #169873
Comments
Since the upstream repo has been archived I've repurposed this issue into adopting a new CSS grammar. |
For folks who are interested in contributing to the CSS grammar: we have adopted new grammars for other languages in the past. The usual formula for adoption is:
|
@alexr00 I am unsure why I would go through the trouble of making an entirely new grammar for CSS, struggle to gain adoption just to have VSCode swoop in when it reaches critical mass? Why can't we do this effort in collaboration within VSCode from the start? Also important to note that this is a self inflicted issue. Forking the Atom css language to the microsoft organization on GitHub seems like the least effort and lowest risk way forward. |
Once something is maintained by the VS Code team then we have to keep maintaining it. At this time we don't have a developer dedicated to maintaining a CSS grammar, which is why we rely on the community. |
But in practice this will most likely be a single individual right? It is also just an impossibly small group of people who fit these characteristics :
Are there volunteers in the VS Code team to help out on a 3rd party package with aspects that do not require in depth knowledge about CSS?
|
I figured y'all would take the atom grammars as they are, adopting / forking them, and then y'all become the new canonical reference we submit grammar updates to. That's why I made a couple PRs to here, seems the easiest route is y'all to pick up the reigns where they got dropped? I'm with Romain here, waiting for some new community driven one is too much work on that maintainer. There's a perfectly good foundation ready to not be an upstream anymore. |
With the exception of writing tests (as we do have some basic grammar tests to keep our grammar integration healthy) these needs are all same reasons that the VS Code team doesn't want to own the grammars. We know maintaining a grammar can be a lot of work and we greatly appreciated all the work that folks do on the OSS projects we depend on for our grammars. https://github.com/jeff-hykin/better-cpp-syntax, https://github.com/fadeevab/make.tmbundle, https://github.com/microsoft/TypeScript-TmLanguage, and https://github.com/jeff-hykin/better-shell-syntax to name just a few. |
We tentatively created a new repo for the old atom grammar where we plan to take fixes: https://github.com/microsoft/vscode-css We're still working out what taking fixes will look like, so any PRs made against that repo may take a while to receive attention. |
Thank you so much @alexr00 for considering a different way forward 🎉 |
@argyleink I would suggest you wait a bit. We don't currently know how we're going to take fixes since we don't have anyone who can review the changes at the moment. |
@argyleink I think we'll soon start merging PRs in https://github.com/microsoft/vscode-css. If you move your PRs there then I will make sure they get reviewed. Thank you for your patience! |
Closing as we have moved the grammar to https://github.com/microsoft/vscode-css |
When authoring multiline CSS strings, e.g.
VSCode's syntax highlighter messes up pretty badly. Here is a real-world example of using a multiline CSS string in a data URL to create a mask for random noise:
Notice how all code after the multiline string is highlighted in yellow.
This issue has already been reported multiple times (#125739, #134642, #146146, #147712, #159730), but all of them have been closed as a duplicate of atom/language-css#181, which through the sunset of Atom has now been closed and archived, too. Creating multiline strings makes things a lot more readable when creating inline SVGs for background images and masks, and currently they are practically unusable in VSCode. I would be greatful if you could prioritize this, especially since I do not believe it to be a very difficult bug to fix.
Tested in VSCode version
1.74.2
(stable) and1.75.0-insider
(pre-release).The text was updated successfully, but these errors were encountered: