Bug 234227
| Summary: | Support CSS d property for SVG | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Bramus <bramus> |
| Component: | SVG | Assignee: | Nobody <webkit-unassigned> |
| Status: | NEW | ||
| Severity: | Normal | CC: | benoit.rouleau, bogdan.brinza, dramaretskiy, graouts, karlcow, maggotfish, ntim, sabouhallawa, tomac, webkit-bug-importer, zimmermann |
| Priority: | P2 | Keywords: | BrowserCompat, InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Bug Depends on: | 276172, 272414, 272415, 272416, 272417, 272446, 272447, 272457, 272507, 272509, 272517, 272621, 274935, 276653 | ||
| Bug Blocks: | 135045, 168519 | ||
Bramus
Currently implemented in Blink/Chromium and soon to be shipped with Gecko/Firefox.
Relevant links:
- SVG Presentation Attributes: https://developer.mozilla.org/en-US/docs/Web/SVG/Attribute/Presentation
- Gecko Intent to Ship: https://groups.google.com/a/mozilla.org/g/dev-platform/c/0hj7fhc2ZGc/m/c3oKiaQ3CwAJ
- Gecko Tracking Bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1744599
- WPT Dashboard: https://wpt.fyi/results/svg/path/property?label=experimental&label=master&aligned
- Demo: https://www.bram.us/2017/06/21/changing-svg-path-data-with-css/
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/86713232>
Karl Dubost
These are tested in https://wpt.fyi/results/svg/path/property?label=experimental&label=master&aligned
WebKit scores 0/235
Karl Dubost
*** Bug 272219 has been marked as a duplicate of this bug. ***
Karl Dubost
Another failure found by Antoine in Bug 272219
https://wpt.fyi/results/web-animations/responsive/d.html
Antoine Quint
The last blocking bug has landed, this is complete as of 277455@main.
Antoine Quint
Some of the blockers got re-opened due to a Speedometer regression, so this is reopened as well.
Tim Nguyen (:ntim)
https://commits.webkit.org/280621@main is the commit that requires a perf investigation.
Benoît Rouleau
This seems to be enabled in TP, should the status be updated?
dramaretskiy
To vouch for the previous comment, is the support stable enough to move out of TP-only?