Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: c672a5df57fceb94daff8177aaa9c9a512526c14
      
https://github.com/WebKit/WebKit/commit/c672a5df57fceb94daff8177aaa9c9a512526c14
  Author: Alan Baradlay <[email protected]>
  Date:   2023-01-05 (Thu, 05 Jan 2023)

  Changed paths:
    A LayoutTests/fast/inline/line-clamp-with-text-align-expected.html
    A LayoutTests/fast/inline/line-clamp-with-text-align.html
    M 
Source/WebCore/layout/formattingContexts/inline/display/InlineDisplayLineBuilder.cpp

  Log Message:
  -----------
  [IFC][line-clamp] Incorrect ellipsis placement when text-align is not "start"
https://bugs.webkit.org/show_bug.cgi?id=249974

Reviewed by Antti Koivisto.

Do not assume that the truncated content is the result of overflowing where
the root inline box always starts at 0.
Move the ellipsis position by the alignment offset (this is specific to line 
clamping).

* LayoutTests/fast/inline/line-clamp-with-text-align-expected.html: Added.
* LayoutTests/fast/inline/line-clamp-with-text-align.html: Added.
* 
Source/WebCore/layout/formattingContexts/inline/display/InlineDisplayLineBuilder.cpp:
(WebCore::Layout::InlineDisplayLineBuilder::trailingEllipsisRect const):

Canonical link: https://commits.webkit.org/258491@main


_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to