1915206 - Try to simplify SVG transformations a bit more.

Status: RESOLVED FIXED

(Core :: SVG, task)

Comment 1

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1915206 - Simplify SVG transforms a bit more. r=longsonr,#layout

We no longer need the concept of "local" transforms after bug 878346.

Local transforms that we'd need in the future could be represented via
attribute mapping.

We still need the viewbox transform shenanigans which applies to
children only, tho.

Comment 2

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1915206 - Make SVG rendering observer container a class flag. r=longsonr,#layout

Drive-by, but it's only set on creation, so no need to make it a state bit.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7acf9bef221f
Simplify SVG transforms a bit more. r=longsonr

Comment 4

[nezriffic]

There is one more issue remaining (I cannot comment on original ticket: https://bugzilla.mozilla.org/show_bug.cgi?id=1906261).

When you use eg. SVGs "circle" - it scales now nicely the whole circles size, but not its stroke.
(See figure #2 and #3 of the attachment, please: https://bugzilla.mozilla.org/attachment.cgi?id=9411197).

Comment 5

[Robert Longson [:longsonr]]

Please create a new bug for that nezriffic. The reason you can't comment on old bugs is to encourage you to raise new ones.

Comment 6

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/7acf9bef221f

Comment 7

[Phabricator Automation]

Comment on attachment 9421076 [details]
Bug 1915206 - Make SVG rendering observer container a class flag. r=longsonr,#layout

Revision D220296 was moved to bug 1915474. Setting attachment 9421076 [details] to obsolete.