Bug 149086 - Whitespace underlines are not repainted on color changes
Summary: Whitespace underlines are not repainted on color changes
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Myles C. Maxfield
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2015-09-11 21:22 PDT by Philip Rogers
Modified: 2024-01-16 17:33 PST (History)
7 users (show)

See Also:


Attachments
Repro for webkit (876 bytes, text/html)
2015-09-11 21:22 PDT, Philip Rogers
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Philip Rogers 2015-09-11 21:22:05 PDT
Created attachment 261042 [details]
Repro for webkit

wkrev.com/150259 introduced a small regression where text decorations such as underlines are not repainted for whitespace text.

A user recently reported this against Chrome and I had merged the above patch a few years ago so we shared this bug. I fixed this by removing the whitespace check which keeps the original optimization but causes whitespace runs to be repainted. My fix landed in https://src.chromium.org/viewvc/blink?revision=202164&view=revision and a similar fix would work in WebKit too.
Comment 1 Myles C. Maxfield 2015-09-12 21:13:56 PDT
This repro is fantastic, thank you!
Comment 2 Brent Fulgham 2022-07-14 14:44:40 PDT
WebKit still gets this wrong, 7 years later.
Comment 3 Radar WebKit Bug Importer 2022-07-14 14:44:49 PDT
<rdar://problem/97036299>
Comment 4 Ahmad Saleem 2022-10-16 10:59:44 PDT
I think this change should be here:

https://github.com/WebKit/WebKit/blob/main/Source/WebCore/rendering/RenderElement.cpp#L319

Replacing "isAllCollapsibleWhitespace" with equivalent of "isBR".
Comment 5 Fujii Hironori 2024-01-16 17:27:44 PST
This is not reproducible with Safari 17.2.1 and STP 186 to me.
Comment 6 Ahmad Saleem 2024-01-16 17:31:55 PST
(In reply to Fujii Hironori from comment #5)
> This is not reproducible with Safari 17.2.1 and STP 186 to me.

Yes - I can't reproduce it as well. Might be fixed between 2022 to 2024 now.
Comment 7 Fujii Hironori 2024-01-16 17:33:30 PST
Thank you for the confirmation. Closed.