Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: d547f9c8eae1f21fe1d88d8e86ac19edaa3f2614
      
https://github.com/WebKit/WebKit/commit/d547f9c8eae1f21fe1d88d8e86ac19edaa3f2614
  Author: Alan Baradlay <za...@apple.com>
  Date:   2025-04-11 (Fri, 11 Apr 2025)

  Changed paths:
    A LayoutTests/fast/ruby/sideways-lr-incorrect-position-expected.html
    A LayoutTests/fast/ruby/sideways-lr-incorrect-position.html
    M Source/WebCore/layout/formattingContexts/inline/InlineContentAligner.cpp
    M 
Source/WebCore/layout/formattingContexts/inline/display/InlineDisplayContentBuilder.cpp
    M 
Source/WebCore/layout/formattingContexts/inline/display/InlineDisplayContentBuilder.h

  Log Message:
  -----------
  Incorrect ruby annotation positioning in sideways-lr
https://bugs.webkit.org/show_bug.cgi?id=291177
<rdar://problem/148713073>

Reviewed by Antti Koivisto.

While BoxGeometry is supposed to hold logical geometry, it is a slightly 
modified one
where left is visual left (required by render tree integration) which means that
WritingMode::isLineOverLeft needs special handling (also see FIXME comment in 
processRubyContent).

* LayoutTests/fast/ruby/sideways-lr-incorrect-position-expected.html: Added.
* LayoutTests/fast/ruby/sideways-lr-incorrect-position.html: Added.
* Source/WebCore/layout/formattingContexts/inline/InlineContentAligner.cpp:
(WebCore::Layout::shiftDisplayBox):
(WebCore::Layout::expandInlineBox):
* 
Source/WebCore/layout/formattingContexts/inline/display/InlineDisplayContentBuilder.cpp:
(WebCore::Layout::InlineDisplayContentBuilder::processRubyContent):
(WebCore::Layout::InlineDisplayContentBuilder::flipLogicalRectToVisualForWritingModeWithinLine
 const): Deleted.
(WebCore::Layout::InlineDisplayContentBuilder::flipRootInlineBoxRectToVisualForWritingMode
 const): Deleted.
* 
Source/WebCore/layout/formattingContexts/inline/display/InlineDisplayContentBuilder.h:
(WebCore::Layout::InlineDisplayContentBuilder::mapInlineRectLogicalToVisual):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to