Bug 183780
| Summary: | height miscalcuation in SVG | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Alastair Campbell <ac> |
| Component: | SVG | Assignee: | Rob Buis <rbuis> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | ahmad.saleem792, bfulgham, rbuis, sabouhallawa, simon.fraser, webkit-bug-importer, zalan, zimmermann |
| Priority: | P2 | Keywords: | InRadar, LayerBasedSVGEngine |
| Version: | Safari 11 | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Alastair Campbell
Zoom on this SVG example has unexpected effects:
https://w3c.github.io/graphics-aria/img/nested-figures.html
(NB: text-size increases work fine, it is the standard pixel-based zoom that is affected.)
It appears that the (inline) y value on the <rect> elements is not increased with the zoom, so the text goes behind the rect.
Noticed whilst @AmeliasBrain was working on the spec: https://w3c.github.io/graphics-aria/#role_other
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Brent Fulgham
Confirmed this is an issue in Safari. Works properly in Chrome and Firefox.
Radar WebKit Bug Importer
<rdar://problem/97099774>
Ahmad Saleem
For future reference, LBSE (Layer Based SVG Engine) does not have height miscalculation issue to make 'text' disappear but it has other issues but still good to note down. Tested using WebKit ToT (266457@main) with LBSE Internal flag turned on.
Rob Buis
(In reply to Ahmad Saleem from comment #3)
> For future reference, LBSE (Layer Based SVG Engine) does not have height
> miscalculation issue to make 'text' disappear but it has other issues but
> still good to note down. Tested using WebKit ToT (266457@main) with LBSE
> Internal flag turned on.
Ahmad, I think this works fine for legacy SVG these days? If so we can add [LBSE] to the bug title for clarity (since there still are problems with it in LBSE for sure).
Ahmad Saleem
(In reply to Rob Buis from comment #4)
> (In reply to Ahmad Saleem from comment #3)
> > For future reference, LBSE (Layer Based SVG Engine) does not have height
> > miscalculation issue to make 'text' disappear but it has other issues but
> > still good to note down. Tested using WebKit ToT (266457@main) with LBSE
> > Internal flag turned on.
>
> Ahmad, I think this works fine for legacy SVG these days? If so we can add
> [LBSE] to the bug title for clarity (since there still are problems with it
> in LBSE for sure).
We have dedicated keyword for issues which also reproduce on LayerBasedSVGEngine, I just added to this bug as well.
Rob Buis
Pull request: https://github.com/WebKit/WebKit/pull/57052
EWS
Committed 306480@main (90a20da0a908): <https://commits.webkit.org/306480@main>
Reviewed commits have been landed. Closing PR #57052 and removing active labels.