[webkit-changes] [WebKit/WebKit] 3931fc: Incorrect caret position in empty elements with a ...

Aditya Keerthi noreply at github.com
Tue Nov 14 08:32:15 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 3931fcef84e408738e94e3a8dea06f611a68b6ec
      https://github.com/WebKit/WebKit/commit/3931fcef84e408738e94e3a8dea06f611a68b6ec
  Author: Aditya Keerthi <akeerthi at apple.com>
  Date:   2023-11-14 (Tue, 14 Nov 2023)

  Changed paths:
    A LayoutTests/editing/selection/caret-in-empty-block-vertical-horizontal-rtl-text-indent-padding-expected.txt
    A LayoutTests/editing/selection/caret-in-empty-block-vertical-horizontal-rtl-text-indent-padding.html
    A LayoutTests/editing/selection/caret-in-empty-block-vertical-rl-rtl-expected.txt
    A LayoutTests/editing/selection/caret-in-empty-block-vertical-rl-rtl.html
    A LayoutTests/editing/selection/caret-in-empty-inline-vertical-rl-rtl-expected.txt
    A LayoutTests/editing/selection/caret-in-empty-inline-vertical-rl-rtl.html
    M Source/WebCore/rendering/CaretRectComputation.cpp
    M Source/WebCore/rendering/RenderBoxModelObject.h
    M Source/WebCore/rendering/RenderBoxModelObjectInlines.h

  Log Message:
  -----------
  Incorrect caret position in empty elements with a vertical writing mode
https://bugs.webkit.org/show_bug.cgi?id=248322
rdar://102652127

Reviewed by Tim Nguyen.

Caret rect computation for empty elements does not fully account for vertical
writing mode. Fix by consistently using logical values.

* LayoutTests/editing/selection/caret-in-empty-block-vertical-horizontal-rtl-text-indent-padding-expected.txt: Added.
* LayoutTests/editing/selection/caret-in-empty-block-vertical-horizontal-rtl-text-indent-padding.html: Added.
* LayoutTests/editing/selection/caret-in-empty-block-vertical-rl-rtl-expected.txt: Added.
* LayoutTests/editing/selection/caret-in-empty-block-vertical-rl-rtl.html: Added.
* LayoutTests/editing/selection/caret-in-empty-inline-vertical-rl-rtl-expected.txt: Added.
* LayoutTests/editing/selection/caret-in-empty-inline-vertical-rl-rtl.html: Added.
* Source/WebCore/rendering/CaretRectComputation.cpp:
(WebCore::computeCaretRectForEmptyElement):
(WebCore::computeCaretRectForBlock):
(WebCore::computeCaretRectForInline):
* Source/WebCore/rendering/RenderBoxModelObject.h:
* Source/WebCore/rendering/RenderBoxModelObjectInlines.h:
(WebCore::RenderBoxModelObject::borderAndPaddingLogicalHeight const):
(WebCore::RenderBoxModelObject::borderAndPaddingLogicalLeft const):
(WebCore::RenderBoxModelObject::borderAndPaddingLogicalRight const):
(WebCore::RenderBoxModelObject::borderAndPaddingEnd const):

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




More information about the webkit-changes mailing list