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

Cursor color should take priority over visual indent guides #8909

Open
mandx opened this issue Nov 25, 2023 · 2 comments
Open

Cursor color should take priority over visual indent guides #8909

mandx opened this issue Nov 25, 2023 · 2 comments
Labels
A-gui Area: Helix gui improvements C-bug Category: This is a bug

Comments

@mandx
Copy link
Contributor

mandx commented Nov 25, 2023

Summary

I was expecting this PR to fix my issue with indent guides coloring, but looks like it doesn't?

I have a custom theme, with just this override (this is the entire contents of the theme file):

inherits = "gruvbox_dark_hard"
"ui.virtual.indent-guide" = "#000000"

Without the ui.virtual.indent-guide override, it looks like this:
image

And with the latest local build (I checked I have this PR already) and the override enabled, it looks like this:
image

The black indent guides look as expected, but I was expecting the cursor (and only the cursor) to look like in the first screenshot.

These are my cursor shape and indent guide settings:

[editor.cursor-shape]
insert = "bar"
normal = "block"
select = "underline"

[editor.indent-guides]
character = ""
skip-level = 1
render = true

Which makes the issue more evident when I switch to Insert mode:
image

The cursor is at the same position as in the second screenshot, but it's effectively invisible.

Reproduction Steps

I tried this:

  1. Create and select a custom theme with just this in it:
inherits = "gruvbox_dark_hard"
"ui.virtual.indent-guide" = "#000000"
  1. Set this in Helix settings:
[editor.cursor-shape]
insert = "bar"
normal = "block"
select = "underline"

[editor.indent-guides]
character = ""
skip-level = 1
render = true

I expected the cursor to look like this when positioned on an indent guide:
image

Instead, it looks like this:
image

Helix log

~/.cache/helix/helix.log ``` [helix.log](https://github.com/helix-editor/helix/files/13464893/helix.log) ```

Platform

Linux

Terminal Emulator

alacritty 0.12.3

Installation Method

source

Helix Version

23.10-75-g8c68074f

@nalply
Copy link

nalply commented Nov 27, 2023

Perhaps related: if a cursor is on a ruler, the cursor and the character disappears.

image

image

image

@mandx
Copy link
Contributor Author

mandx commented Nov 27, 2023

@nalply I believe the case specific to rulers is being tracked here #7371

@kirawi kirawi added the A-gui Area: Helix gui improvements label Dec 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-gui Area: Helix gui improvements C-bug Category: This is a bug
Projects
None yet
Development

No branches or pull requests

3 participants