Bug 268539 - CSS subgrid fails on scroll containers
Summary: CSS subgrid fails on scroll containers
Status: RESOLVED DUPLICATE of bug 267085
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: Safari Technology Preview
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2024-02-01 03:35 PST by Jake Archibald
Modified: 2024-02-08 04:30 PST (History)
7 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jake Archibald 2024-02-01 03:35:15 PST
https://codepen.io/jaffathecake/pen/RwdMgGV

The subgrid applies in Chrome and Firefox, but doesn't seem to apply in Safari (you get a single column).
Comment 1 Radar WebKit Bug Importer 2024-02-01 12:20:48 PST
<rdar://problem/122122998>
Comment 2 Jake Archibald 2024-02-01 13:31:01 PST
To be clear, I haven't managed to confirm which behaviour is expected. I personally expected the Chrome/Firefox behaviour, but I haven't managed to figure the spec out.
Comment 3 Sammy Gill 2024-02-01 13:37:46 PST
Hi Jake,

I believe this issue may have been resolved by https://commits.webkit.org/273134@main

Does this bug reproduce in the iOS 17.4 beta?
Comment 4 Sammy Gill 2024-02-01 13:40:42 PST
(or the macOS 14.4 beta)
Comment 5 Jake Archibald 2024-02-01 14:22:39 PST
I don't have access to betas, but the commit message sounds like it might fix the issue. Cam you test with the demo provided?
Comment 6 Mendel Gordon 2024-02-01 17:59:56 PST
I just tested this in the latest webkit-main and the issue seems to be resolved!
In the current version of safari it's not, so it seems like the changes made to webkit here fixed it, and once that version of webkit is used in safari it should be fixed there as well.
Here is a screenshot of what I see - minibrowser on the left and safari on the right:
https://shottr.cc/s/Vw9i/SCR-20240201-u2p.png
Comment 7 Jake Archibald 2024-02-02 00:30:22 PST
Looks like it's already fixed! Any idea when it might land in Safari? It's not in TP.
Comment 8 Sammy Gill 2024-02-07 15:14:15 PST
According to today's STP release notes this change should be picked up: https://webkit.org/blog/15026/release-notes-for-safari-technology-preview-188/

Please let us know if you can still reproduce the bug even in the latest STP!

*** This bug has been marked as a duplicate of bug 267085 ***
Comment 9 Jake Archibald 2024-02-08 04:30:38 PST
Confirmed! The demo now works perfectly. Thank you!