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

ui.cursor.primary always overrides ui.cursor.insert #1337

Closed
bbodi opened this issue Dec 22, 2021 · 4 comments · Fixed by #5130
Closed

ui.cursor.primary always overrides ui.cursor.insert #1337

bbodi opened this issue Dec 22, 2021 · 4 comments · Fixed by #5130
Labels
A-theme Area: Theme and appearence related C-enhancement Category: Improvements

Comments

@bbodi
Copy link
Contributor

bbodi commented Dec 22, 2021

If ui.cursor.primary is set in a theme, it always overrides e.g. ui.cursor.insert.

E.g. currently in gruvbox theme one can't tell where is the cursor in a selection because it has the same color as the selection:
image

If I change the ui.cursor.primary value of the theme file, the issue is solved:
image

However, I also would like to give ui.cursor.insert a different color, so I know if I am in insert mode, but currently it is not possible

@bbodi bbodi changed the title ui.cursor.primary always overrides `ui.cursor.insert' ui.cursor.primary always overrides ui.cursor.insert Dec 22, 2021
@kirawi kirawi added A-theme Area: Theme and appearence related C-enhancement Category: Improvements labels Dec 23, 2021
@matoous
Copy link
Contributor

matoous commented Dec 27, 2021

I am not sure whether the behavior should be changed 🤔 if the theme specifies different cursor colors for insert/normal mode and doesn't set the primary cursor color this will work fine, maybe we should adjust the theme instead? Or am I missing something?

@bbodi
Copy link
Contributor Author

bbodi commented Dec 28, 2021

Now it is not possible to set a color to normal mode primary cursor, and a different one for insert mode cursor.

@matoous
Copy link
Contributor

matoous commented Dec 29, 2021

But wouldn't that require also having ui.cursor.primary_select and ui.cursor.primary_insert? What should the precedence be, e.g. if I have ui.cursor.primary and ui.cursor.insert if we give precedence to ui.cursor.primary than you can't distinguish between select/insert and other modes. If we give precedence to ui.cursor.insert than we can't distinguish primary vs. additional selections.

@oati
Copy link
Contributor

oati commented Apr 11, 2022

One solution would be to make ui.cursor.insert only apply to the primary cursor when ui.cursor.primary is present.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-theme Area: Theme and appearence related C-enhancement Category: Improvements
Projects
None yet
4 participants