cursor draws behind underline #9015

Open
opened 2026-01-31 01:43:52 +00:00 by claunia · 3 comments
Owner

Originally created by @DHowett on GitHub (Jun 14, 2020).

It is particularly noticeable with the emptyBox cursor.

image

Originally created by @DHowett on GitHub (Jun 14, 2020). It is particularly noticeable with the `emptyBox` cursor. ![image](https://user-images.githubusercontent.com/189190/84587141-8b983680-add1-11ea-83a4-6f73c1e141c5.png)
claunia added the Area-RenderingIssue-BugPriority-3In-PRProduct-Terminal labels 2026-01-31 01:43:52 +00:00
Author
Owner

@zadjii-msft commented on GitHub (Jun 15, 2020):

Oh, cause underlines are drawn on top of the text, in a third pass, aren't they ;___;

This is probably a regression from #6337, isn't it.

I suppose I could still do the post-text pass in PaintCursor. I'm tentatively tossing blocking/1.1 on this, feel free to punt.

@zadjii-msft commented on GitHub (Jun 15, 2020): Oh, cause underlines are drawn on top of the text, in a _third_ pass, aren't they ;___; This is probably a regression from #6337, isn't it. I suppose I could still do the post-text pass in `PaintCursor`. I'm tentatively tossing blocking/1.1 on this, feel free to punt.
Author
Owner

@j4james commented on GitHub (Jun 16, 2020):

If you're looking at possibly changing the render-order here, you may want to think about how this would be impacted by underline/decoration color, should we ever choose to support that. Because if the underline color is different from the text color, I think it should technically be rendered below the text. To make things more complicated, strike-though would still have to be rendered above the text.

I'm not particularly keen on that functionality myself, but just thought it worth mentioning here in case it affects how you want to approach this.

@j4james commented on GitHub (Jun 16, 2020): If you're looking at possibly changing the render-order here, you may want to think about how this would be impacted by underline/decoration color, should we ever choose to support that. Because if the underline color is different from the text color, I think it should technically be rendered _below_ the text. To make things more complicated, strike-though would still have to be rendered _above_ the text. I'm not particularly keen on that functionality myself, but just thought it worth mentioning here in case it affects how you want to approach this.
Author
Owner

@miniksa commented on GitHub (Jun 19, 2020):

Here's another thing to throw in there. Technically we have underline information provided to us by DirectWrite that we're currently ignoring in favor of just drawing a line at the bottom of the box. The DWrite underline information compensates for the stylistic properties of the font. We may want to be using that instead.

@miniksa commented on GitHub (Jun 19, 2020): Here's another thing to throw in there. Technically we have underline information provided to us by DirectWrite that we're currently ignoring in favor of just drawing a line at the bottom of the box. The DWrite underline information compensates for the stylistic properties of the font. We may want to be using that instead.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#9015