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

Theming: Cursor/Selection foreground does not override rendered whitespace. #9825

Closed
levitatingpineapple opened this issue Mar 6, 2024 · 1 comment
Labels
C-bug Category: This is a bug R-duplicate Duplicated issue: please refer to the linked issue

Comments

@levitatingpineapple
Copy link

levitatingpineapple commented Mar 6, 2024

Summary

Currently any colors set on ui.virtual.whitespace are unaffected by colors of the cursor.* and ui.selection

While I see how this might be intentional, I think it has few issues:

  • It does not play well with reversed modifier
  • Any theme that sets both bg and fb won't have cursor or selection visible over rendered whitespace.

Reproduction Steps

Consider this dummy theme:

"ui.selection" = { fg = "red", bg = "blue", modifiers = ["reversed"] }
"ui.virtual.whitespace" = "green"

I expected this to happen the background of the selection would be red:

Screenshot 2024-03-06 at 14 38 55

Instead, the background of the selection is red or green, depending character type:

Screenshot 2024-03-06 at 14 38 03

Platform

macOS

Terminal Emulator

alacritty/iTerm

Installation Method

brew

Helix Version

helix 23.10 (f6021dd)

@levitatingpineapple levitatingpineapple added the C-bug Category: This is a bug label Mar 6, 2024
@kirawi
Copy link
Member

kirawi commented Mar 6, 2024

Duplicate of #5675

@kirawi kirawi marked this as a duplicate of #5675 Mar 6, 2024
@kirawi kirawi closed this as not planned Won't fix, can't repro, duplicate, stale Mar 6, 2024
@kirawi kirawi added the R-duplicate Duplicated issue: please refer to the linked issue label Mar 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: This is a bug R-duplicate Duplicated issue: please refer to the linked issue
Projects
None yet
Development

No branches or pull requests

2 participants