Bug 258505

Summary: Hairline on selection
Product: WebKit Reporter: Ebrahim Byagowi <ebrahim>
Component: TextAssignee: alan <zalan>
Status: NEW    
Severity: Normal CC: mmaxfield, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar
Version: Safari 16   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Safari vs Edge
none
test reduction
none
test reduction none

Ebrahim Byagowi
Reported 2023-06-25 19:03:05 PDT
Created attachment 466821 [details] Safari vs Edge It's similar to bugs like Bug 220548, Reproduction: 1. Open https://the-decoder.com/why-large-ai-language-models-dont-lead-to-human-like-ai/ 2. Select any paragraph from the article Actual: Note that one pixel lines after each word Expected: Similar to other browsers
Attachments
Safari vs Edge (291.74 KB, image/png)
2023-06-25 19:03 PDT, Ebrahim Byagowi 		no flags
Details
test reduction (337 bytes, text/html)
2023-06-26 09:11 PDT, alan 		no flags
Details
test reduction (386 bytes, text/html)
2023-06-26 09:12 PDT, alan 		no flags
Details
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
alan
Comment 1 2023-06-26 09:11:01 PDT
interesting, this seems to be caused by negative word-spacing. Thank you for filing it!
alan
Comment 2 2023-06-26 09:11:26 PDT
Created attachment 466825 [details] test reduction
alan
Comment 3 2023-06-26 09:12:32 PDT
Created attachment 466826 [details] test reduction
Ebrahim Byagowi
Comment 4 2023-06-26 13:25:31 PDT
Thanks for the quick response 😊 There is also a horizontal gap on selection on the second paragraph, maybe that can be interesting on a separate matter as it isn't related to white-space.
Ebrahim Byagowi
Comment 5 2023-06-27 12:13:47 PDT
Please ignore my previous comment as it's a spin off of the main issue here which now it's filed as Bug 258584
alan
Comment 6 2023-06-27 18:37:38 PDT
(In reply to Ebrahim Byagowi from comment #5) > Please ignore my previous comment as it's a spin off of the main issue here > which now it's filed as Bug 258584 Thank you!
alan
Comment 7 2023-06-27 20:28:18 PDT
For simple cases (like this one) we should be able to merge adjacent display boxes so that painting code sees them as one run, but in general we should teach TextBoxPainter to paint selection across runs.
Radar WebKit Bug Importer
Comment 8 2023-07-02 19:04:14 PDT
<rdar://problem/111661724>
Note You need to log in before you can comment on or make changes to this bug.