DxRenderer should eat "concealed" text #14337

Open
opened 2026-01-31 04:07:34 +00:00 by claunia · 3 comments
Owner

Originally created by @carlos-zamora on GitHub (Jun 24, 2021).

Description of the new feature/enhancement

Invisible text is generally rendered as text with a foreground color equal to the background color. This can be repro'd as a part of https://github.com/jdebp/terminal-tests/tree/master/PowerShell).

This makes it not look invisible, however, because we apply the acrylic texture to the background, but not the foreground.

Proposed technical implementation details (optional)

Just make the renderer not render the invisible text. Ignore it completely.

Originally created by @carlos-zamora on GitHub (Jun 24, 2021). # Description of the new feature/enhancement Invisible text is generally rendered as text with a foreground color equal to the background color. This can be repro'd as a part of https://github.com/jdebp/terminal-tests/tree/master/PowerShell). This makes it _not_ look invisible, however, because we apply the acrylic texture to the background, but not the foreground. # Proposed technical implementation details (optional) Just make the renderer _not_ render the invisible text. Ignore it completely.
claunia added the Help WantedArea-RenderingIssue-TaskProduct-Terminal labels 2026-01-31 04:07:34 +00:00
Author
Owner

@j4james commented on GitHub (Jun 25, 2021):

This should perhaps be independent of the concealed attribute though. If someone sets the foreground color to exactly match the background, I would think they'd also expect it to be invisible regardless of acrylic mode wouldn't they?

Also, don't forget about gridlines.

@j4james commented on GitHub (Jun 25, 2021): This should perhaps be independent of the concealed attribute though. If someone sets the foreground color to exactly match the background, I would think they'd also expect it to be invisible regardless of acrylic mode wouldn't they? Also, don't forget about gridlines.
Author
Owner

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

Huh. Grid lines . . . good catch 😄

@DHowett commented on GitHub (Jun 28, 2021): Huh. Grid lines . . . good catch :smile:
Author
Owner

@DHowett commented on GitHub (Jul 2, 2021):

Triaged into Terminal v2 -- help wanted. Thanks for all the input here!

@DHowett commented on GitHub (Jul 2, 2021): Triaged into Terminal v2 -- help wanted. Thanks for all the input here!
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#14337