| Summary: | [LBSE] Add 'viewBox' support | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Nikolas Zimmermann <zimmermann> |
| Component: | SVG | Assignee: | Nikolas Zimmermann <zimmermann> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | rbuis, sabouhallawa, webkit-bug-importer, zimmermann |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=241763 | ||
| Bug Depends on: | |||
| Bug Blocks: | 90738, 242833 | ||
|
Description
Nikolas Zimmermann
2022-06-19 14:31:43 PDT
Pull request: https://github.com/WebKit/WebKit/pull/1622 Pull request: https://github.com/WebKit/WebKit/pull/1623 (In reply to Nikolas Zimmermann from comment #1) > Pull request: https://github.com/WebKit/WebKit/pull/1622 Please ignore that PR, 1623 is correct, I made a mistake with the branch naming convention before. NOTE: The original bug description is obsolete: compositing layers are no longer involved with viewBox/zoom/pan handling. It's just too wasteful, despite the good intentions... I'm now wrapping all children of the outermost <svg> in an anonymous RendrSVGViewportContainer that carries the viewBox/zoom/pan induced transformation as regular CSS transform: greatly simplifies the overall code, no special cases for SVG viewBox necessary (vs. LBSE downstream, that had hacked viewBox support into RenderLayer). Committed 252643@main (c6915835241e): <https://commits.webkit.org/252643@main> Reviewed commits have been landed. Closing PR #1623 and removing active labels. |