Skip to content

Fix invalid color customization #224272

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

Closed
wants to merge 3 commits into from

Conversation

lewohy
Copy link

@lewohy lewohy commented Jul 30, 2024

There seems to be an incorrect setting in "editor.colorcustomization"

This is how the settings for "editorCursor.background" and "editorCursor.foreground" used to work.
before
"editorCursor.background" became the color of the letter and "editorCursor.foreground" became the background color.

And I changed it to look like this.
after

@aeschli aeschli assigned alexdima and unassigned aeschli Aug 6, 2024
@alexdima
Copy link
Member

You are correct. The color names don't make sense / are swapped. I could trace this back to this commit: 74f5073#diff-e5004af9f54ea471fd25b0c1bebd990b520ab0a32932b8256c0d5739b663b2d2 . Before that commit, there was a single color, called editorCursor, which was the background color and the foreground color was derived from it.


Unfortunately it is too late to fix this mistake now, as this would break countless themes, including the built-in ones:

CleanShot.2025-07-15.at.02.23.22.mp4

We will have to live with this misnomer.

@alexdima alexdima closed this Jul 14, 2025
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

Successfully merging this pull request may close these issues.

3 participants