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

Fix text in terminal showing as bold when dimmed #7491

Merged

Conversation

clo4
Copy link
Contributor

@clo4 clo4 commented Feb 7, 2024

Fixes text in the terminal displaying as bold when it's actually just dim. I think it was just a simple oversight because the original code |'s together the BOLD and DIM_BOLD flags, which is the same as DIM_BOLD, which is wrong because it should only be BOLD :p

Release Notes:

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Feb 7, 2024
@clo4
Copy link
Contributor Author

clo4 commented Feb 7, 2024

Hmm. Dim text is also just not dimmed at all. Going to fix that too.

@clo4
Copy link
Contributor Author

clo4 commented Feb 7, 2024

Nevermind, out of scope, going to fix that separately. Ready for review

Copy link
Contributor

@mikayla-maki mikayla-maki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you!

@mikayla-maki mikayla-maki merged commit 8f7d786 into zed-industries:main Feb 8, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed The user has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Terminal displays dim text as bold
2 participants