Binaryify / OneDark-Pro

Atom's iconic One Dark theme for Visual Studio Code
https://marketplace.visualstudio.com/items?itemName=zhuangtongfa.Material-theme
MIT License
1.52k stars 289 forks source link

help comments are bold!!! #242

Closed PhoenixNil closed 6 years ago

PhoenixNil commented 6 years ago

Hello, developer, I hope that you can change the C/C++ comment to bold instead of italic, because under Windows, the Chinese italic display is too ugly, can you change it back? :)

chpxu commented 6 years ago

@PhoenixNil Hey there, it is most likepy something due to my pull request, I am. working on a bunch of fixes, is it possible you coupd send me a screenshot, and I'll see if I can fix anything (note, I am not the official dev as you can tell just noting that I may have broken a bunch of stuff when they merged my pull)

Thanks!

PhoenixNil commented 6 years ago

default

I hope they become bold rather than italic ;( ,italic is very ugly

chpxu commented 6 years ago

Yes I do agree with you, unfortunately it seems we can't do character specific changes, ie right now it isn't possible to fix it just for Chinese characters, even if we did just change it overall, many people may or may not like it, and we've yet to get feedback so we'll see how it goes, thanks for some screenshots though, will experiment!

Binaryify commented 6 years ago

I just change it @PhoenixNil @beastdestroyer

rossnichols commented 6 years ago

@Binaryify, bold doesn't look that great, and also the change doesn't affect / / style comments either - those are still italic. If you're moving away from italic (which I'd be sad about, I like the italic), can you at least not make it bold? Just make it gray.

Can themes have settings? Ideally this could just be customized.

Binaryify commented 6 years ago

@rossnichols here is the docs https://binaryify.github.io/OneDark-Pro/#/

rossnichols commented 6 years ago

@Binaryify, I already edited it locally, but I presume my change will be wiped out next time you push an update.

chpxu commented 6 years ago

@rossnichols It will, you can submit your changes as a pull request or wait until @Binaryify has done it himself (if he chooses to do so)

Binaryify commented 6 years ago

@rossnichols I mean you can custom it on your setting.json no in the theme.json

chpxu commented 6 years ago

Crap Im stupid, I have never used that so I completely forgot about that... Ops :>

chpxu commented 6 years ago

Also @Binaryify Im going to open an issue on this, but units (px, em, rem etc.) aren't red

rossnichols commented 6 years ago

@Binaryify, thanks, I added this to revert the change:

"editor.tokenColorCustomizations": {
    "textMateRules": [
        {
            "scope": "comment.line.double-slash.c,comment.block.documentation.c,comment.line.double-slash.cpp,comment.block.documentation.cpp",
            "settings": {
                "fontStyle": "italic"
            }
    ]
}

But couldn't you have just recommended to @PhoenixNil to do this in the first place? Comments have been italic in this theme since I started using it months ago.

Binaryify commented 6 years ago

@rossnichols you are right

Binaryify commented 6 years ago

@PhoenixNil I just change it back on version 2.15.4 to make comments more unified you need add this snippet into your settings now

"editor.tokenColorCustomizations": {
    "textMateRules": [
        {
            "scope": "comment.line.double-slash.c,comment.block.documentation.c,comment.line.double-slash.cpp,comment.block.documentation.cpp",
            "settings": {
                "fontStyle": "bold"
            }
    ]
}
stone-zeng commented 6 years ago

I want to mention that, some Chinese fonts, e.g. Sarasa Gothic, are specially designed for programming and thus have italic shape. Hope you may enjoy it.