Skip to content

Git Bash bold in One Half Light runs text together #16168

Closed

Description

Windows Terminal version

1.17.11461.0

Windows build number

10.0.19045.0

Other Software

git version 2.42.0.windows.2

Steps to reproduce

I'm using Git Bash with the "One Half Light" color scheme. I want to show something in bold, like this:

echo "Uploading $(tput bold)some-really-long-filename.zip$(tput sgr0) to the server ..."

At first the bold didn't show up any differently than the regular text. Then I went to "Text Formatting" and changed "Intense text style" to "Bold font". Now bold shows up. But often it runs the text together.

Expected Behavior

I wanted bold text, without running the text together. Here is how it looks (as expected) in the terminal of VS Code.

vs-code-bold

Actual Behavior

Microsoft Terminal jams some of the words together. Note the space after ….zip completely disappeared.

microsoft-termina-bold

The point of using bold was to make certain things stand out, but this bug defeats the purpose by jumbling things together.

The output of Maven, for example, is a bit garbled, because many of the goal identifiers are shoved over, "eating" the space of the words after them.

This only seems to happen of the run of bold text is sufficiently long.

My guess off the top of my head is that the bold algorithm is calculating the space taken up based upon the non-bold text, and not taking into account that the bold text is wider. Over a few characters this per-letter error accumulates. Just a hunch.

Maybe you could just add an option to use the VS Code terminal renderer in Microsoft Terminal?

Activity

  1. added
    Issue-BugIt either shouldn't be doing this or needs an investigation.
    Needs-TriageIt's a new issue that the core contributor team needs to triage at the next triage meeting
    on Oct 13, 2023
  2. garretwilson commented on Oct 13, 2023

    @garretwilson
    Author

    This happens on the "Solarized Light" color scheme as well. Must be a general Microsoft Terminal rendering bug.

  3. garretwilson commented on Oct 13, 2023

    @garretwilson
    Author

    Unfortunately rendering bold as "Bright colors" (the default" doesn't seem to make any difference at all, so it looks like I'll have to abandon Microsoft Terminal and use VS Code terminal for almost everything until this is fixed. I guess if I need two directories open I can just add more terminal tabs in VS Code.

  4. zadjii-msft commented on Oct 13, 2023

    @zadjii-msft
    Member

    I think this was fixed in 1.18 or 1.19 - Can you double check in Terminal Preview/?

  5. garretwilson commented on Oct 13, 2023

    @garretwilson
    Author

    I think this was fixed in 1.18 or 1.19

    Good news! Thanks for letting me know.

    Can you double check in Terminal Preview/?

    Unfortunately I just don't have the time at the moment, but I'll be happy to check and report back once those versions are released and my installation gets upgraded.

  6. added
    Needs-AttentionThe core contributors need to come back around and look at this ASAP.
    and removed
    Needs-Author-FeedbackThe original author of the issue/PR needs to come back and respond to something
    on Oct 13, 2023
  7. zadjii-msft commented on Oct 13, 2023

    @zadjii-msft
    Member

    Good news! You can actually just immediatly install the update by downloading the .msixbundle from our releases page:

  8. added
    Needs-Author-FeedbackThe original author of the issue/PR needs to come back and respond to something
    and removed
    Needs-AttentionThe core contributors need to come back around and look at this ASAP.
    on Oct 13, 2023
  9. garretwilson commented on Oct 20, 2023

    @garretwilson
    Author

    I think this was fixed in 1.18 or 1.19

    It's not fixed in v1.18.2822.0 at least.

  10. added
    Needs-AttentionThe core contributors need to come back around and look at this ASAP.
    and removed
    Needs-Author-FeedbackThe original author of the issue/PR needs to come back and respond to something
    No-Recent-ActivityThis issue/PR is going stale and may be auto-closed without further activity.
    on Oct 20, 2023
  11. added
    Needs-Author-FeedbackThe original author of the issue/PR needs to come back and respond to something
    and removed
    Needs-AttentionThe core contributors need to come back around and look at this ASAP.
    on Oct 25, 2023
  12. zadjii-msft commented on Oct 25, 2023

    @zadjii-msft
    Member

    Ah shoot - can you check again and make sure that the new text renderer is enabled? I forgot that it's disabled by default in the Stable builds (but is enabled by default in preview)

    image

  13. garretwilson commented on Oct 25, 2023

    @garretwilson
    Author

    Woohoo! 🎉 That was it! I needed to turn on "Use the new text renderer".

    Very nice to see a bug getting fixed. Thanks!

  14. added
    Needs-AttentionThe core contributors need to come back around and look at this ASAP.
    and removed
    Needs-Author-FeedbackThe original author of the issue/PR needs to come back and respond to something
    on Oct 25, 2023
  15. DHowett commented on Oct 26, 2023

    @DHowett
    Member

    Amazing, thanks for confirming! That new engine will eventually be made the default - we're currently collecting reliability (and bug fix!) data on it. Let us know if you find anything weird!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Issue-BugIt either shouldn't be doing this or needs an investigation.Needs-AttentionThe core contributors need to come back around and look at this ASAP.Needs-TriageIt's a new issue that the core contributor team needs to triage at the next triage meeting

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions