Add support for SGR 2 Faint #9308

Closed
opened 2026-01-31 01:51:21 +00:00 by claunia · 9 comments
Owner

Originally created by @willmcgugan on GitHub (Jun 27, 2020).

Originally assigned to: @DHowett on GitHub.

We do not support Faint text.

Original content

Description of the new feature/enhancement

Windows Terminal doesn't appear to support the dim ansi escape code (SGR 2) or italic (SGR 3). Adding support for these would I think bring WT up to feature parity with most OSX and Linux terminals.

There are other SGR escape codes that would be nice to have, but aren't as common or as useful as dim and italic.

Proposed technical implementation details (optional)

Dim strikes me as an easy one to achieve. I imagine italic would be a little trickier.

Originally created by @willmcgugan on GitHub (Jun 27, 2020). Originally assigned to: @DHowett on GitHub. We do not support Faint text. <details> <summary>Original content</summary> # Description of the new feature/enhancement Windows Terminal doesn't appear to support the *dim* ansi escape code (SGR 2) or *italic* (SGR 3). Adding support for these would I think bring WT up to feature parity with most OSX and Linux terminals. There are other SGR escape codes that would be nice to have, but aren't as common or as useful as dim and italic. # Proposed technical implementation details (optional) Dim strikes me as an easy one to achieve. I imagine italic would be a little trickier. </details>
Author
Owner

@DHowett commented on GitHub (Jun 28, 2020):

Thanks!

This is a curious issue. We're tracking all manner of different versions of "terminal doesn't render specific attributes":

  • #109 enable bold (this is a very divisive issue), #5384 "should bold apply to ITU indexed colors", #5682 "is bold 1;39 a separate color?"
  • #2916 support parsing faint/a couple others
  • #5461 italics aren't being rendered

I just started tackling #109, despite it being divisive, and I think the infrastructure we need to build to get it off the ground (a "base + variant" system for fonts) is going to be required for this.

I'm going to close this shortly but rework all of the above issues into an amalgamated "we should support different attributes, from parsing through to rendering" issue. Does that sound good?

@DHowett commented on GitHub (Jun 28, 2020): Thanks! This is a curious issue. We're tracking all manner of different versions of "terminal doesn't render specific attributes": * #109 enable bold (this is a very divisive issue), #5384 "should bold apply to ITU indexed colors", #5682 "is bold 1;39 a separate color?" * #2916 support parsing faint/a couple others * #5461 italics aren't being rendered I just started tackling #109, despite it being divisive, and I think the infrastructure we need to build to get it off the ground (a "base + variant" system for fonts) is going to be required for this. I'm going to close this shortly but rework all of the above issues into an amalgamated "we should support different attributes, from parsing through to rendering" issue. Does that sound good?
Author
Owner

@willmcgugan commented on GitHub (Jun 28, 2020):

Sounds good to me 👍

@willmcgugan commented on GitHub (Jun 28, 2020): Sounds good to me 👍
Author
Owner

@csdvrx commented on GitHub (Jul 3, 2020):

I'm going to close this shortly but rework all of the above issues into an amalgamated "we should support different attributes, from parsing through to rendering" issue. Does that sound good?

I added #6779 which provides a test script and an expected output.

It links another issue: there is no font switching, which would be useful for mixing different fonts.

@csdvrx commented on GitHub (Jul 3, 2020): > I'm going to close this shortly but rework all of the above issues into an amalgamated "we should support different attributes, from parsing through to rendering" issue. Does that sound good? I added #6779 which provides a test script and an expected output. It links another issue: there is no font switching, which would be useful for mixing different fonts.
Author
Owner

@j4james commented on GitHub (Jul 9, 2020):

@DHowett I've just realised this has been assigned to you, and wanted to check exactly which parts you're working on, or planning to work on. Reason being I was about to start on the faint attribute, as a follow up to the color refactoring in PR #6853, but if you already have plans for that, I can leave it to you.

Edit: I should add that my intention was to treat the faint attribute as entirely separate from bold/bright - i.e. both can be applied at the same time. My reasoning for this is explained in issue https://github.com/microsoft/terminal/issues/2916#issuecomment-652061927.

@j4james commented on GitHub (Jul 9, 2020): @DHowett I've just realised this has been assigned to you, and wanted to check exactly which parts you're working on, or planning to work on. Reason being I was about to start on the faint attribute, as a follow up to the color refactoring in PR #6853, but if you already have plans for that, I can leave it to you. Edit: I should add that my intention was to treat the faint attribute as entirely separate from bold/bright - i.e. both can be applied at the same time. My reasoning for this is explained in issue https://github.com/microsoft/terminal/issues/2916#issuecomment-652061927.
Author
Owner

@DHowett commented on GitHub (Jul 9, 2020):

@j4james it’s assigned to me for administrative reasons; I’m going to create a scenario for the various attributes impacting the display of text, and then close/rework some of the ones we already have open to fit under it. If you want to do work in this area, you’re welcome to 😄 I just may change which issue number your PR eventually closes

@DHowett commented on GitHub (Jul 9, 2020): @j4james it’s assigned to me for administrative reasons; I’m going to create a scenario for the various attributes impacting the display of text, and then close/rework some of the ones we already have open to fit under it. If you want to do work in this area, you’re welcome to :smile: I just may change which issue number your PR eventually closes
Author
Owner

@DHowett commented on GitHub (Jul 13, 2020):

I've repurposed this to track Faint, and crossed out mention of Faint in #2916. Your pull request can unabashedly close this issue and drop mention of 2916.

@DHowett commented on GitHub (Jul 13, 2020): I've repurposed this to track Faint, and crossed out mention of Faint in #2916. Your pull request can unabashedly close this issue and drop mention of 2916.
Author
Owner

@DHowett commented on GitHub (Jul 13, 2020):

I've filed #6879 to track "all" attributes.

@DHowett commented on GitHub (Jul 13, 2020): I've filed #6879 to track "all" attributes.
Author
Owner

@ghost commented on GitHub (Jul 22, 2020):

:tada:This issue was addressed in #6873, which has now been successfully released as Windows Terminal Preview v1.2.2022.0.🎉

Handy links:

@ghost commented on GitHub (Jul 22, 2020): :tada:This issue was addressed in #6873, which has now been successfully released as `Windows Terminal Preview v1.2.2022.0`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v1.2.2022.0) * [Store Download](https://www.microsoft.com/store/apps/9n8g5rfz9xk3?cid=storebadge&ocid=badge)
Author
Owner

@DHowett commented on GitHub (Aug 21, 2020):

🎉 As of Windows Insider build 20197, this is also supported in the traditional console.

@DHowett commented on GitHub (Aug 21, 2020): 🎉 As of Windows Insider build 20197, this is also supported in the traditional console.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#9308