jonschlinkert / sublime-monokai-extended

Extends Monokai from Soda with additional syntax highlighting for Markdown, LESS, HTML, Handlebars and more.
https://github.com/jonschlinkert
MIT License
514 stars 91 forks source link

Why the color is different from before? #58

Closed guozhenyi closed 8 years ago

guozhenyi commented 8 years ago

Yesterday, when I'm coding, I found the color is changed. It's a unexpected change.

before after

jonschlinkert commented 8 years ago

Sorry about that, I'll look into it and see what happened.

karolyi commented 8 years ago

+1 for python source highlighting (and probably more too): original Monokai: image

Monokai Extended: image

def and class keywords are in different color. more to come, probably ...

ParadiseDS commented 8 years ago

+1 for php highlighting The monokai bright seems to be the origin one, but it cannot highlight under diff syntax (probably more...)

jonschlinkert commented 8 years ago

Yeah it shouldn't have changed. I'll revert whatever caused the issue.

karolyi commented 8 years ago

Package Control has upgraded Monokai Extended to 2015.12.29.22.12.53

the syntax is still not fixed

OmeGak commented 8 years ago

+1 for Python syntax highlight

gerardroche commented 8 years ago

PHP too.

At least one of the offending pr's is https://github.com/jonschlinkert/sublime-monokai-extended/pull/52

A diff between https://github.com/jonschlinkert/sublime-monokai-extended/commit/e99726ebe785dfb6352f175dd7decb363c3a05b0 and https://github.com/jonschlinkert/sublime-monokai-extended/commit/9a99355f6561c83a566b701ca30f41806c55b098

diff

Reverting causes some big conflicts.

karolyi commented 8 years ago

Package Control: Upgraded Monokai Extended to 2016.01.31.22.05.34

It seems the problem is solved. thx!

OmeGak commented 8 years ago

Keeps looking wrong in Python in 2016.01.31.22.05.34.

screenshot from 2016-02-01 11 21 48

karolyi commented 8 years ago

Interesting, mine is fixed:

image

karolyi commented 8 years ago

According to the latest merged pull request, the version installed must have fixed the issue:

https://github.com/jonschlinkert/sublime-monokai-extended/commit/6de4a667aa915cad33b8d1ec239138db42b2ca99

I suggest you doublecheck your settings.

OmeGak commented 8 years ago

Thanks! I realized I was using the automatically converted Monokai Extended from Sublime Linter instead, which doesn't get updated automatically. I simply had to select a color scheme from Monokai Extended ones and Sublime Linter overwrote the old one.

jonschlinkert commented 8 years ago

I'm going to close since it sounds like this is fixed. However, we can reopen if others report that it's not. thanks for the feedback

ice9js commented 8 years ago

PHP syntax highlighting still looks like it's broken to me.

jonschlinkert commented 8 years ago

Still? Or as a result of https://github.com/jonschlinkert/sublime-monokai-extended/issues/62?

gerardroche commented 8 years ago

I just checked against build 3103 back to 3095 and it is still messed up. It's broken until back to commit https://github.com/jonschlinkert/sublime-monokai-extended/commit/e99726ebe785dfb6352f175dd7decb363c3a05b0 which was ok.

jonschlinkert commented 8 years ago

ok good to know, thanks for the detail

gerardroche commented 8 years ago

This is still an issue, at least for PHP.

There are several issues.

One is that, from what I understand, the theme is generated from the .JSON-tmTheme files. I wasn't aware of this when I contributed a patch so I didn't modify those JSON files and generate the theme from them, I don't know how to do that. I edited the .tmTheme file directly.

In a later pull request my changes were overwritten, as I think many other changes were, see https://github.com/jonschlinkert/sublime-monokai-extended/pull/52

That pull request says "adds support for ~~ strike-through text" but look at the massive changes it made. It changed all kinds of things. For example, any previous changes that didn't use the JSON file to generate the theme were lost.

Look at the JSON files modifications in that pull request, they only add changes to the markup strikout, e.g.:

 +            "name": "Markup: strike", 
+            "scope": "markup.strike", 
+            "settings": {
+                "fontStyle": "", // Or add `strike`, but it doesn't support
+                "foreground": "#cc4273"
+            }
+        }, 
+        {

Then look at the changes to the tmTheme files, big sweeping changes.

Am I correct in thinking that is what caused the issue?