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

cursor draws behind underline #6499

Open
DHowett opened this issue Jun 14, 2020 · 3 comments
Open

cursor draws behind underline #6499

DHowett opened this issue Jun 14, 2020 · 3 comments
Labels
Area-Rendering Text rendering, emoji, complex glyph & font-fallback issues In-PR This issue has a related PR Issue-Bug It either shouldn't be doing this or needs an investigation. Priority-3 A description (P3) Product-Terminal The new Windows Terminal.
Milestone

Comments

@DHowett
Copy link
Member

DHowett commented Jun 14, 2020

It is particularly noticeable with the emptyBox cursor.

image

@DHowett DHowett added Area-Rendering Text rendering, emoji, complex glyph & font-fallback issues Issue-Bug It either shouldn't be doing this or needs an investigation. Product-Terminal The new Windows Terminal. Priority-3 A description (P3) labels Jun 14, 2020
@ghost ghost added the Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting label Jun 14, 2020
@zadjii-msft
Copy link
Member

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 zadjii-msft added Severity-Blocking We won't ship a release like this! No-siree. and removed Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting labels Jun 15, 2020
@zadjii-msft zadjii-msft added this to the Terminal v1.1 milestone Jun 15, 2020
@j4james
Copy link
Collaborator

j4james commented 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.

@miniksa
Copy link
Member

miniksa commented 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 miniksa removed the Severity-Blocking We won't ship a release like this! No-siree. label Jul 13, 2020
@zadjii-msft zadjii-msft modified the milestones: Terminal Backlog, Backlog Jan 4, 2022
@microsoft-github-policy-service microsoft-github-policy-service bot added the In-PR This issue has a related PR label Mar 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Rendering Text rendering, emoji, complex glyph & font-fallback issues In-PR This issue has a related PR Issue-Bug It either shouldn't be doing this or needs an investigation. Priority-3 A description (P3) Product-Terminal The new Windows Terminal.
Projects
None yet
Development

No branches or pull requests

5 participants