Bug 232176
| Summary: | Improve performance of nested grids | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Tim Nguyen (:ntim) <ntim> |
| Component: | Layout and Rendering | Assignee: | Nobody <webkit-unassigned> |
| Status: | NEW | ||
| Severity: | Normal | CC: | agafvv, ahmad.saleem792, bfulgham, cathiechen, dobiasch, maggotfish, mattwoodrow, mendelgordons, michael.zacharov, rbuis, rego, rik, sgill26, simon.fraser, svillar, tobias47n9e, webkit-bug-importer, zalan, zsun |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| See Also: |
https://bugs.webkit.org/show_bug.cgi?id=125145 https://bugs.webkit.org/show_bug.cgi?id=268595 https://bugs.webkit.org/show_bug.cgi?id=271083 |
||
Tim Nguyen (:ntim)
See: http://exponential-layout.glitch.me
From: https://developer.chrome.com/blog/layoutng/
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/84823492>
Ulrich Dobiasch
Safari 16 – still present – 15+ nested grids and the cpu is stalling
Michael
Safari 17 – still present – 15+ nested grids and the cpu is stalling
Mendel Gordon
See https://web.archive.org/web/20211008182612/https://developer.chrome.com/blog/layoutng/ since the original link is dead
(In reply to Tim Nguyen (:ntim) from comment #0)
> See: http://exponential-layout.glitch.me
>
> From: https://developer.chrome.com/blog/layoutng/
tobias47n9e@gmail.com
Just a comment regarding importance:
Would be nice to see this get higher priority. Grid is used for so many layout problems now, and the developer pain of having to remove all that just for Webkit based browsers is incredible.
Ahmad Saleem
I tested this post 276633@main and now this test can be pushed to 100+ without any issue on M1 Pro with 16 GB.
Change progressing this - https://commits.webkit.org/276633@main
@Sammy - should we mark this duplicate of other bug?
Manuel Rego Casasnovas
(In reply to Ahmad Saleem from comment #6)
> I tested this post 276633@main and now this test can be pushed to 100+
> without any issue on M1 Pro with 16 GB.
>
> Change progressing this - https://commits.webkit.org/276633@main
>
> @Sammy - should we mark this duplicate of other bug?
I can confirm this behavior, without that patch my computer starts to struggle around adding the 15th layer (or earlier). With that patch I've managed to add >150 layers easily.