Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 90dfd6f5fe81463f1301a253764fc76e434fb1d7
      
https://github.com/WebKit/WebKit/commit/90dfd6f5fe81463f1301a253764fc76e434fb1d7
  Author: Sammy Gill <[email protected]>
  Date:   2026-09-01 (Tue, 01 Sep 2026)

  Changed paths:
    M LayoutTests/TestExpectations
    A 
LayoutTests/imported/w3c/web-platform-tests/css/css-grid/grid-lanes/item-placement/flow-tolerance/column-fit-tolerance-infinite-explicit-placement-expected.html
    A 
LayoutTests/imported/w3c/web-platform-tests/css/css-grid/grid-lanes/item-placement/flow-tolerance/column-fit-tolerance-infinite-explicit-placement-ref.html
    A 
LayoutTests/imported/w3c/web-platform-tests/css/css-grid/grid-lanes/item-placement/flow-tolerance/column-fit-tolerance-infinite-explicit-placement.html
    A 
LayoutTests/imported/w3c/web-platform-tests/css/css-grid/grid-lanes/item-placement/row-auto-placement-003-expected.html
    A 
LayoutTests/imported/w3c/web-platform-tests/css/css-grid/grid-lanes/item-placement/row-auto-placement-003-ref.html
    A 
LayoutTests/imported/w3c/web-platform-tests/css/css-grid/grid-lanes/item-placement/row-auto-placement-003.html
    M Source/WebCore/rendering/GridLanesLayout.cpp

  Log Message:
  -----------
  [Grid Lanes] Explicitly-placed items should not advance the auto-placement 
cursor.
https://bugs.webkit.org/show_bug.cgi?id=322873
rdar://problem/186118858

Reviewed by Elika Etemad.

Unlike regular grid, where all explicitly-placed items are positioned before 
auto-placement
runs, Grid Lanes places items interleaved in order. That raised the question of 
whether an
explicitly-placed item should move the auto-placement cursor along with it.

The CSSWG resolved this in
https://github.com/w3c/csswg-drafts/issues/13987#issuecomment-5192487721:

    RESOLVED: Remove issue, leave spec as-is (not updating auto-placement 
cursor with
    explicit placement)

GridLanesLayout was updating m_autoFlowNextCursor from 
insertIntoGridAndLayoutItem(), which
runs for every item, so explicitly-placed items dragged the cursor along too. 
Move the
update up into placeGridLanesItems(), which already knows whether the item 
resolved a
definite grid-axis position, and only advance the cursor for auto-placed items.

* 
LayoutTests/imported/w3c/web-platform-tests/css/css-grid/grid-lanes/item-placement/flow-tolerance/column-fit-tolerance-infinite-explicit-placement-expected.html:
 Added.
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-grid/grid-lanes/item-placement/flow-tolerance/column-fit-tolerance-infinite-explicit-placement-ref.html:
 Added.
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-grid/grid-lanes/item-placement/flow-tolerance/column-fit-tolerance-infinite-explicit-placement.html:
 Added.
Added some new tests that make sure this behavior is correct when fit-tolerance 
is infinite.

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications

Reply via email to