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

Default can be bold #416

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Default can be bold #416

wants to merge 1 commit into from

Conversation

ryuukk
Copy link

@ryuukk ryuukk commented Apr 6, 2024

Fix #415

@randy3k
Copy link
Owner

randy3k commented Aug 17, 2024

Though it doesn't work if the background is white and foreground is black.

@ryuukk
Copy link
Author

ryuukk commented Aug 17, 2024

What would be the proper fix?

@randy3k
Copy link
Owner

randy3k commented Aug 18, 2024

There is no concept of a brighter foreground though as foreground is not necessarily black or white. Not sure if it could be fixed.

@ryuukk
Copy link
Author

ryuukk commented Aug 19, 2024

There is no concept of a brighter foreground though as foreground is not necessarily black or white. Not sure if it could be fixed.

nothing "can't be fixed" when they are wrong, you just have to do the right thing

if other terminals do it properly

there is no reason Terminus can't do it too, it's supposed to do what a terminal do

@randy3k
Copy link
Owner

randy3k commented Aug 19, 2024

Sublime Text doesn't have a proper way to allow us to dynamically bold a region, so we could only support brighten bold text. However, unlike the canonical ANSI colors, there is no brighter color of the default foreground. It is actually correct for not "bolding" the default color.

I think the only wrong thing is that Terminus doesn't support bold, but as I have mentioned above, it is due to Sublime Text limitation. If we are willing to assume that the foreground is always the ANSI color number 8 (i.e, white), we could use ANSI color number 16 (i.e. bright white) to "bold" the foreground. That assumption doesn't always hold, though.

@frou
Copy link
Contributor

frou commented Aug 20, 2024

Could the choice of which colour to use for such text be delegated to the Terminus themes? Add an additional property to the JSON and let the theme designer (or the user configuring their "user" Terminus theme) decide which of the colours to map it onto.

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.

Bright code without color specified results in no bright text
3 participants