-
Notifications
You must be signed in to change notification settings - Fork 148
Atom 1.32 breaks syntax highlighting #281
Comments
Thanks for contributing! Can you please share some pictures or other describe in further detail the differences in syntax highlighting that you are experiencing? |
Thanks @mskornilov for posting pics of the changed syntax highlighting behavior. Mine looks the same. |
Sorry now that I think about it, which syntax highlighting style (1.32 w/ tree-sitter or 1.31.2 without) is the correct one? Maybe it was broken all this time and now is fixed in 1.32? |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
It's definitely the Tree Sitter Parsers that causing the syntax highlighting problem, and in multiple languages. In Atom 1.32 if you go to Settings -> Core -> Tree Sitter Parsers and uncheck it highlighting behavior returns to normal. I'm actually kind of surprised something like this wasn't caught by someone before 1.32 rolled out. |
This comment has been minimized.
This comment has been minimized.
The highlighting of keyword arguments has been fixed and the fix will ship in Atom 1.32.1. |
the language highlithing do not deal correctly with greek letters (python 3 fully support unicode characters), so insted of [a-zA-Z] better can be [a-zA-Zα-ωΑ-Ω] |
I'm going to close this out since the original issue has been fixed. @asiloisad I apologize for breaking the highlighting for identifiers with greek letters. That has been fixed (though the fix has not shipped yet). @TheElementalOfCreation - The TODO issue is tracked at atom/language-todo#82. The issue where we were highlighting an erroneous Everyone, if there are other issues with python highlighting, please open a new issue that describes the problem specifically. Thank you! |
Atom 1.32.2 and 1.33.0-beta2 are out. Sorry for the delay, folks. |
Sorry want to ask again for some feedback... since I've been using Atom with Tree Sitter Parsers off for so long I am now unsure what would be the correct way Python should be highlighted :-/ Since I wrote my previous comment March 7, Atom or other updates have improved the Tree Sitters Python syntax highlighting to be more correct now. There still some differences so I'm guessing it was always wrong when Tree Sitters was off (and I just got used to it)? |
@hermidalc function parameters and subclasses will be highlighted again starting in (I think) Atom 1.37.0 thanks to #297 and #298 from @caleb531.
Other variables being highlighted was removed in the tree-sitter grammar as there was no way to highlight all variables consistently (and, indeed, you'll notice the inconsistency in your own examples even with tree-sitter turned off). |
@hermidalc Just an update: my improvements to function/class parameter highlighting in #297 and #298 will be actually released in Atom v1.38.0—they are already in beta0, in fact—per the official release notes. If you want them sooner, you can clone my fork of language-python and rebuild apm by running the following:
When Atom 1.38.0 is officially released, you can later unlink and rebuild again to use the bundled language-python instead of my fork:
Then restart Atom to complete the process. |
@caleb531 highlighting in 1.38 looking good now. From the above two tree sitters on/off examples I had above the only possible missing hightlight now is in function def if you have e.g. *args or **kwargs the "args" and "kwargs" parts are not orange. |
That was handled in #303 and should be coming down the release pipeline next :). |
Please follow these instructions already posted if you run into further issues, thanks! |
Prerequisites
Description
Python syntax highlighting is not working after update to Atom 1.32
Steps to Reproduce
Expected behavior: Proper syntax highlighting
Actual behavior: Parts of code which should be highlighted are now broken and are white
Reproduces how often: 100%
Versions
Atom 1.32
The text was updated successfully, but these errors were encountered: