Bug 5461 – Text width measured incorrectly when text-align: justify

• Status: RESOLVED
• Resolution: FIXED
• Component: Layout and Rendering

[Duncan Wilcox]

Load attached sample document, hold cmd-ctrl and hit "d" (keep holding cmd-ctrl), now hovering over 
words pops up the dictionary lookup. Hover towards ends of lines, the highlited text isn't exactly 
aligned with the word beneath it.

The only strange thing in the sample document is the "text-align: justify".

More specifically the highlighted text appears to be aligned on the non justified word position.

Tested with freshly checked out tree, this is printed to the console:

=================
ERROR: TEXTINPUT: characterIndexForPoint: not yet implemented
(/Users/duncan/builds/WebKit/WebView.subproj/WebHTMLView.m:4874 -[WebHTMLView
(WebNSTextInputSupport) characterIndexForPoint:])
=================

[Duncan Wilcox]

Created attachment 4442 [details]
test case

[Alexey Proskuryakov]

Justified text also has serious problems when being edited in Blot and Mail; I think the root cause is the 
same. To reproduce, open the test case in Blot and try moving the caret with arrow keys or mouse clicks - 
its positioning doesn't account for the justification. Renaming the bug and bumping priority to P2.

BTW, an implementation for characterIndexForPoint is currently in the commit queue (bug 5230), and it 
doesn't help in this case.

[Alexey Proskuryakov]

Created attachment 4443 [details]
Automated test case

[Eric Seidel (no email)]

Particularly "note" and "are" in the second paragraph.

Created attachment 5353 [details]
Work in progress

Created attachment 5376 [details]
Use WebTextRenderer selection rects

There's more to this patch than just fixing the bug: past fixes to "highlight
leaves trails" were about narrowing WebTextRenderer's highlight rect to fit
into InlineTextBox's selectionRect, which due to the "rounding hack" was
narrower than AppKit's highlight rect for the same text. By moving all
selection/highlight rect computations to WebTextRenderer, this patch eliminates
unnecessary intermediate rounding and gives wider rects that match AppKit more
closely. I think it also improves caret positioning in RTL text.

It looks like AppKit uses different rounding modes for the right hand side of
the highlight (ceil) and for the caret (round). This patch sacrifices highlight
to get caret right. I guess this can be improved in the future.

[Darin Adler]

Comment on attachment 5376 [details]
Use WebTextRenderer selection rects

Nice. r=me

[Daniel Bates]

Committed in <https://trac.webkit.org/changeset/11846/>.