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

[Bug] Setting just the selected text colour without the background text colour will cause the background colour to change to cyan #1020

Closed
1 task done
ClementTsang opened this issue Feb 18, 2023 · 0 comments · Fixed by #1021
Assignees
Labels
bug Something isn't working the way that is expected.

Comments

@ClementTsang
Copy link
Owner

Checklist

Describe the issue

The default background colour is light blue. However, if you set just the foreground selected text colour, it will change to cyan.

This is due to having multiple sources of truth in our default styling - see here where it's set to Color::Cyan, but here by default uses Color::LightBlue.

What operating system and version are you using?

Arch Linux

What architecture are you using?

x86_64 / AMD64

What terminal(s) are you running bottom on?

kitty 0.27.1, but irrelevant

What version of bottom are you running?

0.8.0

How did you install bottom?

Arch repos.

How can we reproduce this?

Use a config file of:

[colors]
selected_text_color="Red"

Additional information

No response

@ClementTsang ClementTsang added the bug Something isn't working the way that is expected. label Feb 18, 2023
@ClementTsang ClementTsang self-assigned this Feb 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working the way that is expected.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant