Branch: refs/heads/main Home: https://github.com/WebKit/WebKit Commit: 48936aafec37b3b17c182f210f46e65bf6df9ba9 https://github.com/WebKit/WebKit/commit/48936aafec37b3b17c182f210f46e65bf6df9ba9 Author: Ahmad Saleem <ahmad.saleem...@gmail.com> Date: 2022-09-17 (Sat, 17 Sep 2022)
Changed paths: A LayoutTests/fast/text/flexbox-selection-gap-painting-expected.html A LayoutTests/fast/text/flexbox-selection-gap-painting.html A LayoutTests/fast/text/grid-selection-gap-painting-expected.html A LayoutTests/fast/text/grid-selection-gap-painting.html M Source/WebCore/rendering/RenderBlock.cpp Log Message: ----------- Text Selection on flex and grid box items does not work as expected Text Selection on flex and grid box items does not work as expected https://bugs.webkit.org/show_bug.cgi?id=119878 Reviewed by Alan Bujtas. Merge & Extend for Grid - https://src.chromium.org/viewvc/blink?view=revision&revision=171810 Webkit paints "selection gaps" that extend the selection to the bounds of the selection root. If the selection spans two flex or grid items, and the selection root is the RenderBody, the selection gap for the first item is painted on top of the second item, obscuring its text. By making each flex or grid item a selection root, the selection gap is painted only up to the bounds of the flex item. This is the logic we use for table cells and positioned objects. * Source/WebCore/rendering/RenderBlock.cpp: (RednerBlock::isSelectionRoot) - Added isFlexItemIncludingDeprecated and isGridItem to behave similar for Selection paint as Table * LayoutTests/fast/text/flexbox-selection-gap-painting.html: Added Test * LayoutTests/fast/text/flexbox-selection-gap-painting-expected.html: Added Test Expectations * LayoutTests/fast/text/grid-selection-gap-painting.html: Added Test * LayoutTests/fast/text/grid-selection-gap-painting-expected.html: Added Test Expectations Canonical link: https://commits.webkit.org/254602@main _______________________________________________ webkit-changes mailing list webkit-changes@lists.webkit.org https://lists.webkit.org/mailman/listinfo/webkit-changes