-
-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Render foreground rulers (┊) by default
#14453
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
base: master
Are you sure you want to change the base?
Conversation
cee5915 to
483eff2
Compare
archseer
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm so how do I retain the original behaviour on my personal config? I prefer no character and theming the background color so that I can see the ruler across all lines, including those that overlap text. e.g. if I'm looking at unformatted text then with this new change the ruler will barely be visible
|
You can get the old ruler style back by setting the ruler char to space and using a background color in a theme. Setting it to background in the default theme would be a bit unfortunate though since it would mean that the default char Could you try out this branch for a while and see how it feels? I like that |
|
As part of #12369 I have been using |
|
I'll give |
483eff2 to
2304f42
Compare
¦) by default┊) by default
|
Ok after trying it for a while I agree, |
Make ruler char configurable, default to `┊` Use foreground rulers in all current themes
Make ruler char configurable, default to `┊` Use foreground rulers in all current themes
Make ruler char configurable, default to `┊` Use foreground rulers in all current themes
Make ruler char configurable, default to `┊` Use foreground rulers in all current themes

This is a continuation of #11798 / #9256. Before this patch rulers are drawn by theming the ruler's cell. With this patch
¦characters (by default) are drawn on any cells without other text.Before this patch the guidance for themes is to set
ui.virtual.ruleras a background color so that a vertical bar of cells have a noticeable color. With this this change it's the opposite: you should setui.virtual.ruleras a foreground color to theme the¦s. Recommending setting background colors prior to this patch is awkward because all other keys inui.virtual.*typically theme the foreground. And it's easy to forget to theme it if you don't often use rulers.This change updates all current themes to use foreground rulers. It's a naive change though. I expect that some themes will need follow-up changes to the colors.