Skip to content
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

Backslashes became very hard to see (almost invisible) #577

Closed
realnc opened this issue Mar 23, 2018 · 12 comments
Closed

Backslashes became very hard to see (almost invisible) #577

realnc opened this issue Mar 23, 2018 · 12 comments

Comments

@realnc
Copy link

realnc commented Mar 23, 2018

After I upgraded to 1.205, all backslashes in my code became very hard to spot. Basically, this:

auto msg = "Test\nFoo bar.\n";

looks more like this:

auto msg = "Test nFoo bar. n";

The backslashes are very faint and of the wrong color. This is a problem only when using a black background with light text (which is what I use in my IDE, which is Qt Creator.)

For comparison, this is with DejaVu:

1

It has good visibility. But with Fira Code it's hard to read:

2

@j-f1
Copy link
Contributor

j-f1 commented Mar 23, 2018

This is an intentional change (see the changelog, third bullet point)

@realnc
Copy link
Author

realnc commented Mar 23, 2018

How do I prevent it? This makes the font unusable for me :-/

@tonsky
Copy link
Owner

tonsky commented Mar 24, 2018

I plan to change how it works in the next version. For now you can use previous font version (1.204)

@doughsay
Copy link

doughsay commented Apr 4, 2018

For what it's worth, I really like the new thin backslash, and it doesn't look bad at all in my editor.

screen shot 2018-04-04 at 4 42 51 pm

@miniBill
Copy link

miniBill commented Apr 10, 2018

I think that a lot depends on whether your editor highlights escape codes (in which case the thin backslash is good) or not

@Moulick
Copy link

Moulick commented Apr 22, 2018

Yep, the thin backslash is hard to read in Python as well.

@tonsky tonsky closed this as completed in 9b28330 Sep 30, 2018
@doughsay
Copy link

😢

@j-f1
Copy link
Contributor

j-f1 commented Feb 21, 2019

Would it be possible to bring this back for just \\? It makes sequences of slashes much easier to read.

@tonsky
Copy link
Owner

tonsky commented Feb 21, 2019

Maybe—please create a separate issue for that

@j-f1
Copy link
Contributor

j-f1 commented Feb 21, 2019

#720

@Vampire
Copy link
Contributor

Vampire commented Sep 3, 2019

Very sad this was removed, I really liked seeing an escaping backslash thin, showing that it is an escape sequence. :-(

@j-f1
Copy link
Contributor

j-f1 commented Sep 3, 2019

Could this be a stylistic set @tonsky?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants