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

Add additional text styles (dim, overline, double underline, curvy underline, blink, & blink fast) #1630

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

Conversation

joouha
Copy link
Contributor

@joouha joouha commented May 8, 2022

This PR implements additional text styles:

  • Dim
  • Overline
  • Double underline
  • Curvy underline
  • Blink
  • Blink fast
Screenshot

extra-styles webp

@jonathanslenders
Copy link
Member

Thanks!

Before merging this it is also important to benchmark the performance impact on various platforms. Both CPU and memory. This data structure is initialized many times, and it would be a shame to merge this if it significantly impacts the performance on various systems, for a feature that's only available in a few terminals.

@ryran
Copy link

ryran commented May 6, 2023

Man I reeeeally want this... "dim" support especially. Sigh. Would this make ANSI() support the corresponding raw ansi codes (like dim)? Or would that need to be a separate bit of work altogether?

@joouha joouha force-pushed the extra-style-attributes branch 2 times, most recently from 0795e9f to 030ce8e Compare January 24, 2024 23:19
- Dim
- Overline
- Double underline
- Curvy underline
- Blink
- Blink fast
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.

3 participants