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

improved default color scheme #7971

Closed
wants to merge 1 commit into from

Conversation

singh-jagjot
Copy link

The default color scheme of alacritty has 'color.normal.black' as "#181818" which is the same as 'color.primary.background' due to which text with 'normal.black' color is impossible to view. This pull request fixes that issue by making the value of 'normal.color' to '#333333' (just a lighter shade). This will improve the experience of default color scheme users.

Before:
Screenshot 2024-05-12 at 1 19 42 PM

After:
Screenshot 2024-05-12 at 1 20 05 PM

@chrisduerr chrisduerr closed this May 12, 2024
@singh-jagjot
Copy link
Author

@chrisduerr Is there any specific reason for closing this? Just curious. I believe this is an improvement, text in the terminal is visible to a certain extent after this fix. I know this is just a minor change, but one can use the default color scheme without any hiccups. 🙂

@chrisduerr
Copy link
Member

I'm not accepting any colorscheme changes since that topic is far too subjective.

@singh-jagjot
Copy link
Author

@chrisduerr Thanks for the reply. I do agree with you and I'm not trying to put my colorscheme above others. It's just one color. This issue kept me searching for fixes. Without this fix, the only possible solution to this will be to use another theme or edit alacritty.toml manually. Because the background and normal.black color is same in default color scheme, you cannot read the terminal in case text is of normal.black color. If you think this is still unnecessary, that's fine by me. 🙂

@kchibisov
Copy link
Member

Because the background and normal.black color is same in default color scheme, you cannot read the terminal in case text is of normal.black color. If you think this is still unnecessary, that's fine by me. 🙂

it's like that in 99% of the themes. Just use bright black, a lot of software which uses regular one does it in bold to request bright, for whatever reason. Better to just patch such software to use bright black as they want to.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants