De-underline hyperlink on hover-out, even if not focused #12239

Closed
opened 2026-01-31 03:09:59 +00:00 by claunia · 6 comments
Owner

Originally created by @PhMajerus on GitHub (Jan 28, 2021).

Testing WT Preview 1.6.10272.0.

Ok, now that we got the underline hyperlink on hover, even when not focused (#8475, #8615)... can we have them always de-underline on mouseout as well? You know, for consistency? 😉

Right now, the underline gets removed only when another hyperlink gets underlined, not when the mouse moves away from it.

Originally created by @PhMajerus on GitHub (Jan 28, 2021). Testing WT Preview 1.6.10272.0. Ok, now that we got the underline hyperlink on hover, even when not focused (#8475, #8615)... can we have them always de-underline on mouseout as well? You know, for consistency? 😉 Right now, the underline gets removed only when another hyperlink gets underlined, not when the mouse moves away from it.
Author
Owner

@zadjii-msft commented on GitHub (Jan 29, 2021):

YUP we should do that, good catch.

@zadjii-msft commented on GitHub (Jan 29, 2021): YUP we should do that, good catch.
Author
Owner

@Don-Vito commented on GitHub (Jan 30, 2021):

@zadjii-msft - I am wondering why we chose to implement #8475 rather than #6459. Won't it make the behavior more consistent and prevent issues like this one?

And it is also a one liner 😄

@Don-Vito commented on GitHub (Jan 30, 2021): @zadjii-msft - I am wondering why we chose to implement #8475 rather than #6459. Won't it make the behavior more consistent and prevent issues like this one? And it is also a one liner :smile:
Author
Owner

@DHowett commented on GitHub (Feb 4, 2021):

@don-vito curious though -- would implementing focus-follows-mouse fix the issue where hyperlinks get underlined and never get un-underlined when the whole window is out of focus?

FFM is very controversial. I don't think it's the solution to any problems except the problem some users perceive where we do not have FFM. 😄

@DHowett commented on GitHub (Feb 4, 2021): @don-vito curious though -- would implementing focus-follows-mouse fix the issue where hyperlinks get underlined and never get un-underlined _when the whole window is out of focus?_ FFM is very controversial. I don't think it's the solution to any problems except the problem some users perceive where we do not have FFM. :smile:
Author
Owner

@Don-Vito commented on GitHub (Feb 4, 2021):

@don-vito curious though -- would implementing focus-follows-mouse fix the issue where hyperlinks get underlined and never get un-underlined when the whole window is out of focus?

FFM is very controversial. I don't think it's the solution to any problems except the problem some users perceive where we do not have FFM. 😄

@DHowett - what you mention regarding the entire window losing focus is another aspect that requires a fix, but it is not related directly to this report. The problem here is due to a gap in the implementation of mouse hover over unfocused pane. And the fix here will be different from the fix that will be required for window losing focus.

My suggestion for FFM is replacing the code that introduced this gap. I was not aware though that FFM is so controversial 😊

@Don-Vito commented on GitHub (Feb 4, 2021): > @don-vito curious though -- would implementing focus-follows-mouse fix the issue where hyperlinks get underlined and never get un-underlined _when the whole window is out of focus?_ > > FFM is very controversial. I don't think it's the solution to any problems except the problem some users perceive where we do not have FFM. :smile: @DHowett - what you mention regarding the entire window losing focus is another aspect that requires a fix, but it is not related directly to this report. The problem here is due to a gap in the implementation of mouse hover over unfocused pane. And the fix here will be different from the fix that will be required for window losing focus. My suggestion for FFM is replacing the code that introduced this gap. I was not aware though that FFM is so controversial :blush:
Author
Owner

@Don-Vito commented on GitHub (Feb 4, 2021):

I am handing this separately from FFM.

@Don-Vito commented on GitHub (Feb 4, 2021): I am handing this separately from FFM.
Author
Owner

@ghost commented on GitHub (Feb 11, 2021):

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

Handy links:

@ghost commented on GitHub (Feb 11, 2021): :tada:This issue was addressed in #9039, which has now been successfully released as `Windows Terminal Preview v1.6.10412.0`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v1.6.10412.0) * [Store Download](https://www.microsoft.com/store/apps/9n8g5rfz9xk3?cid=storebadge&ocid=badge)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#12239