Feature Request: Swapping the foreground color and the background color

[zhou13]

Is your feature request related to a problem? Please describe.
Many terminals support to swap the fg and bg color as the color under the cursor. However, I search through the document and I cannot find an example of this feature. Such a method is better than hardcode the cursor color as it is able to reflect the original color under the cursor.

Describe the solution you'd like
In alacritty, I am able to configure like this:

  cursor:
    text: CellBackground
    cursor: CellForeground

In wezterm, I hope I will be able to do things like

return {
  colors = {
      cursor_bg = "fg",
      cursor_fg = "bg",
  }
}

Describe alternatives you've considered
N./A.
Additional context
N./A.

[wez]

Are you looking specifically for the ability to configure colors in terms of the foreground/background, or would you be satisfied with a separate option that overrides the cursor color setting and forces it into reverse video mode?

[zhou13]

It would be the latter one, as in the reverse video mode.

[wez]

in the next nightly build (in about 20-30 mins), you can set force_reverse_video_cursor = true

[zhou13]

It seems that force_reverse_video_cursor = true no longer works for me: the cursor color still applies.

[wez]

It works for me with a quick test here; could you open a new bug issue with all the information needed to reproduce the issue?

[zhou13]

I take a look at the code and I think there is some misunderstanding. When the setting is enabled, it simply uses palette.background to replace palette.cursor_fg. This is not useful, as I can directly set palette.cursor_fg to be the color in palette.background. The intended results should be swaping the fg and bg color of the character under the cursor, rather than choosing from a predetermined palette.

Alacritty result:

wezterm result:

[AtifChy]

@zhou13 how do you enable dynamic cursor color on wezterm?

[wez]

I just pushed a fix for this; it should show up in the nightly builds in ~30 minutes.

@AtifChy https://wezfurlong.org/wezterm/config/lua/config/force_reverse_video_cursor.html

[zhou13]

Looks like it works correctly. Thank you!

[github-actions]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.