Bug 183780 – height miscalcuation in SVG

• Status: RESOLVED
• Resolution: FIXED
• Component: SVG

[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

[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.