Bug 268539
| Summary: | CSS subgrid fails on scroll containers | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Jake Archibald <jaffathecake> |
| Component: | CSS | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED DUPLICATE | ||
| Severity: | Normal | CC: | bfulgham, mattwoodrow, mendelgordons, sgill26, simon.fraser, webkit-bug-importer, zalan |
| Priority: | P2 | Keywords: | InRadar |
| Version: | Safari Technology Preview | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Jake Archibald
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).
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/122122998>
Jake Archibald
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.
Sammy Gill
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?
Sammy Gill
(or the macOS 14.4 beta)
Jake Archibald
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?
Mendel Gordon
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
Jake Archibald
Looks like it's already fixed! Any idea when it might land in Safari? It's not in TP.
Sammy Gill
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 ***
Jake Archibald
Confirmed! The demo now works perfectly. Thank you!