Created attachment 462754 [details] Firefox 105 OK, Safari 16 FAIL. Resizes causes the page to crash and reload. I am working on a page which leverages subgrid to align some various items. I noticed that the content was offset by about the same size as 2x the column-gap value I was using. The grid is modified using a custom property replacing a single column with a `repeat` that divides the space up into 5 columns within that central "content area". Additionally, the page crashes when I resize the window (see the bottom of the attached screenshot), I suspect the complexity of all the custom properties determining the size as well as the subgrid being calculated on the fly, all wrapped in a media query, is too much to handle at once. While my original page crashes pretty consistently, the demo seems to be quite slower but not always crashing. However, my page has many more elements placed more specifically along the grid, whereas the demo is simpler so the issue might not be as severe I have reproduced the issue on a small sample, DEMO: https://codepen.io/chriskirknielsen/pen/dyeeRYY Tested: Firefox 105: PASS Safari 16: FAIL Safari TP 154: FAIL
Created attachment 462768 [details] rendering in safari, firefox, chrome Tested on macOS 13.0 --- Safari Technology Preview 16.0 18615.1.6.2 (STP 154) Firefox Nightly 107.0a1 10722.9.30 Google Chrome Canary 108.0.5336.0 5336.0 3 different renderings.
(In reply to Karl Dubost from comment #1) > Google Chrome Canary 108.0.5336.0 5336.0 Is that with subgrid support enabled?
(In reply to Tim Nguyen (:ntim) from comment #2) > Is that with subgrid support enabled? In chrome://flags/ there is nothing to enable subgrid. (Version 108.0.5338.0 (Official Build) canary (arm64))
(In reply to Karl Dubost from comment #3) > (In reply to Tim Nguyen (:ntim) from comment #2) > > Is that with subgrid support enabled? > > In chrome://flags/ there is nothing to enable subgrid. > (Version 108.0.5338.0 (Official Build) canary (arm64)) Pretty sure you need --enable-blink-feature=LayoutNGSubgrid as a command line argument still.
<rdar://problem/100939450>
I can't reproduce any crashes here, but I do see a bug with gap. It looks like there's not great interop here (I see differences in handling with Chrome too, even with subgrid support enabled). I think Firefox is right for your test case though. Will try fix and add a few new WPT tests for this.
Pull request: https://github.com/WebKit/WebKit/pull/6193
Committed 256621@main (87d76d7dac68): <https://commits.webkit.org/256621@main> Reviewed commits have been landed. Closing PR #6193 and removing active labels.