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

Change dark_plus inlay-hints colors to more pleasant colors #7611

Merged
merged 2 commits into from
Jul 12, 2023
Merged

Change dark_plus inlay-hints colors to more pleasant colors #7611

merged 2 commits into from
Jul 12, 2023

Conversation

mkblast
Copy link
Contributor

@mkblast mkblast commented Jul 12, 2023

A little change to the dark_plus theme.

  • Before:
    2023-07-12T17:48:17,965895541+01:00

  • After:
    2023-07-12T17:48:02,745411692+01:00

@David-Else
Copy link
Contributor

David-Else commented Jul 12, 2023

I think you meant Dark Plus theme and this title is wrong.

@mkblast Did you check out #6270 ?

I agree that they look dreadful at the moment, I think they were hastily done to copy the VS Code default.. but in VS Code there are also other ways it is styled so it looks better. If a direct copy of the colors from the official theme are needed, maybe better to take them from the dark+ experimental version as discussed in the discussion? What do you think? Your colors look OK to me, but I think some others will want to input their opinion on this.

@mkblast mkblast changed the title Changing code_dark inlay-hints colors to more pleasent colors. Changing dark_plus inlay-hints colors to more pleasent colors. Jul 12, 2023
@mkblast
Copy link
Contributor Author

mkblast commented Jul 12, 2023

I think a background colors is not needed. for me its too distracting since it has more contrast.

However, I like the foreground color of dark+ experimental version way better than what i choose.

image

@David-Else
Copy link
Contributor

However, I like the foreground color of dark+ experimental version way better than what i choose.

So do I, it would be cool to change your PR to that colour, it is then an official VS Code colour too :)

@mkblast
Copy link
Contributor Author

mkblast commented Jul 12, 2023

Are we good with no background color or we wait for more input?

@the-mikedavis the-mikedavis changed the title Changing dark_plus inlay-hints colors to more pleasent colors. Change dark_plus inlay-hints colors to more pleasant colors Jul 12, 2023
@pascalkuthe pascalkuthe merged commit e86bb64 into helix-editor:master Jul 12, 2023
mtoohey31 pushed a commit to mtoohey31/helix that referenced this pull request Jun 2, 2024
…itor#7611)

* Changing code_dark inlay-hints colors.

* Using dark_plus_experimental inlay hints is better ;)
smortime pushed a commit to smortime/helix that referenced this pull request Jul 10, 2024
…itor#7611)

* Changing code_dark inlay-hints colors.

* Using dark_plus_experimental inlay hints is better ;)
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.

5 participants