Open
Bug 1393421
Opened 7 years ago
Updated 2 years ago
Clean up pocket-animation.svg
Categories
(Firefox :: Pocket, enhancement, P4)
Firefox
Pocket
Tracking
()
NEW
Tracking | Status | |
---|---|---|
firefox57 | --- | affected |
People
(Reporter: jaws, Unassigned)
References
(Blocks 1 open bug)
Details
(Whiteboard: [reserve-photon-animation])
Some comments from bug 1386029, (In reply to Jonathan Watt [:jwatt] from comment #10) > BTW pocket-animation.svg has a lot of unnecessary overhead. > > * all the filters are identical - we could just use one > > * there's no need to put the <filter> in a <defs> > > * in fact it looks like the filter is just about coloring - it > could be replaced by passing in a context-stroke value as a second > color which would render much faster and with less CPU work than a > filter > > * all the <svg> (other than the root) and <g> elements are > unnecessary > > * the 'transform' attributes could be applied to the path data > statically to avoid having to transform all the paths at runtime > which is more expensive (In reply to Jonathan Watt [:jwatt] from comment #11) > (In reply to Jonathan Watt [:jwatt] from comment #10) > > * all the <svg> (other than the root) and <g> elements are > > unnecessary > > To be clear the 'x' attributes on the inner-<svg> elements would need to be > propagated into the path data.
Flags: qe-verify-
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•