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

sorbet: cursor is hard to see in visual mode #258

Open
micampe opened this issue Feb 21, 2024 · 5 comments
Open

sorbet: cursor is hard to see in visual mode #258

micampe opened this issue Feb 21, 2024 · 5 comments

Comments

@micampe
Copy link

micampe commented Feb 21, 2024

It's difficult to see the cursor when in visual mode because Cursor uses reverse and Visual foreground is very similar to the Normal background.

Screenshot 2024-02-21 at 09 59 57

I worked around this by setting fixed cursor colors instead of using reverse, like I've seen it's being done to solve a similar issue in MatchParen.

Screenshot 2024-02-21 at 10 07 10

I'm using MacVim r179 (Vim 9.1.0)

@lifepillar
Copy link
Contributor

Another option is to set a blinking cursor in Visual mode. Something like this:

set guicursor+=v:blinkwait100-blinkon400-blinkoff400

@micampe
Copy link
Author

micampe commented Feb 22, 2024

oh actually blinking seems to be the default, but I disabled all blinking in my gvimrc.

micampe added a commit to micampe/dotfiles that referenced this issue Feb 22, 2024
@neutaaaaan
Copy link
Collaborator

I'm not sure what could really be done here : blinking is the default behaviour of every terminal emulator I've used, and of nearly every glass tty I've seen.
The implementation can differ (reverse the cell properly or use the terminal's cursor color) but as far as I know there's little that can be done on our end that wouldn't risk overlapping in strange ways with the end user's terminal emulator settings.

@micampe
Copy link
Author

micampe commented Feb 22, 2024

To clarify, this is in MacVim, not in a terminal emulator (though I did disable blinking there too).

Using fixed colors for the cursor, like other themes in this repository do, seems to fix the problem, not sure if it introduces other regressions.

I'm not pushing for this to be changed, I am fine with keeping my workaround if this is determined to be an appropriate default. I have other small adjustments I apply to this and other schemes, I just thought this one was worth reporting.

I did notice while testing that quiet behaves the same.

@neutaaaaan
Copy link
Collaborator

I did notice while testing that quiet behaves the same.

My colorschemes are in effect built on top of quiet. and to an extent this might be a side effect of quiet's original focus on surviving borked environments against all odds, let alone sanity.

I see retrobox and zaibatsu use a fixed cursor color and don't use reverse for visual selections. I'll try and look into this next weekend.

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

No branches or pull requests

3 participants