Implement path() on `d` property on SVG path element
Categories
(Core :: SVG, enhancement, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox91 | --- | fixed |
People
(Reporter: 709922234, Assigned: boris)
References
(Blocks 2 open bugs, )
Details
(Keywords: dev-doc-complete, Whiteboard: [layout:backlog], [wptsync upstream])
Attachments
(12 files, 1 obsolete file)
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
image/png
|
Details |
Comment 2•8 years ago
|
||
Updated•7 years ago
|
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 3•5 years ago
|
||
This should be addressed in https://github.com/w3c/svgwg/issues/320. Not merged yet, so let's wait.
Assignee | ||
Comment 4•5 years ago
|
||
To be clear, d
property/attribute doesn't support path()
function, so Gecko cannot render the path correctly.
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Updated•5 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Comment 5•4 years ago
|
||
Now that we triage by severity, setting this bug's priority to P2 to represent near-term backlog status. See https://wiki.mozilla.org/Platform/Layout#Backlog_Tracking_in_Bugzilla
Updated•4 years ago
|
Assignee | ||
Comment 6•4 years ago
|
||
Assignee | ||
Comment 7•4 years ago
|
||
We parse the attribute and store into CSS.
Updated•3 years ago
|
Updated•3 years ago
|
Assignee | ||
Comment 8•3 years ago
|
||
Use the style value when building the SVG path and its display items.
Besides, we should also invalid the cache when there are any CSS updates.
Assignee | ||
Comment 9•3 years ago
|
||
Assignee | ||
Comment 10•3 years ago
|
||
Assignee | ||
Comment 11•3 years ago
|
||
Assignee | ||
Comment 12•3 years ago
|
||
Assignee | ||
Comment 13•3 years ago
|
||
Unfortunately, we still have to convert the cpp version into rust version.
However, it's still better than parsing it again.
Assignee | ||
Comment 14•3 years ago
|
||
Now we have d property, so the list of auto completion should include d
property when typing 'd'. We need to add more tabs or up/down keys to
choose the keyword we want.
Assignee | ||
Comment 15•3 years ago
|
||
Updated•3 years ago
|
Updated•3 years ago
|
Assignee | ||
Comment 16•3 years ago
|
||
This will be merged into part 3 after getting r+.
Assignee | ||
Comment 17•3 years ago
|
||
When using a mpath elememt which linked to a external path element, we use
GetDistancesFromOriginToEndsOfVisibleSegments() to do a quick check if we
really need to build the path (note: see
SVGMotionSMILAnimationFunction::RebuildPathAndVerticesFromMpathElem),
so we have to get the d property value from style in this function.
Updated•3 years ago
|
Updated•3 years ago
|
Updated•3 years ago
|
Assignee | ||
Comment 18•3 years ago
|
||
Comment 19•3 years ago
|
||
Comment 21•3 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/8e60664181ba
https://hg.mozilla.org/mozilla-central/rev/44fd66676b8c
https://hg.mozilla.org/mozilla-central/rev/8ae9761a40ee
https://hg.mozilla.org/mozilla-central/rev/22a2ccbad39c
https://hg.mozilla.org/mozilla-central/rev/a1b8cf1afda3
https://hg.mozilla.org/mozilla-central/rev/1321ace60cdd
https://hg.mozilla.org/mozilla-central/rev/f3fd6d964758
https://hg.mozilla.org/mozilla-central/rev/b6d681d6d66d
https://hg.mozilla.org/mozilla-central/rev/b26fb23539f7
https://hg.mozilla.org/mozilla-central/rev/4ee3b8eb9c61
https://hg.mozilla.org/mozilla-central/rev/52f82029a1e5
Comment 22•3 years ago
|
||
Hooray! Happy to see this fixed :)
Comment 25•3 years ago
|
||
This doesn't appear fixed, or maybe there's another bug?
https://codepen.io/therealmarcone/pen/OJjeBpw draws the path in Chrome, but not in Firefox 94.0.2 nor in 78.15.0esr.
Assignee | ||
Comment 26•3 years ago
|
||
(In reply to marco.nelissen from comment #25)
This doesn't appear fixed, or maybe there's another bug?
https://codepen.io/therealmarcone/pen/OJjeBpw draws the path in Chrome, but not in Firefox 94.0.2 nor in 78.15.0esr.
Did you enable the preference (i.e. layout.css.d-property.enabled
)? I saw a path inside the button. The preference is enabled on only Nightly channel now.
Assignee | ||
Comment 27•3 years ago
|
||
The result on Nightly for the test in comment 25.
Comment 28•3 years ago
|
||
No, I didn't enable the preference, because I didn't realize that was necessary in order to actually enable a bugfix from 6 months ago in a build from 2 days ago. Will it ever be enabled by default?
Assignee | ||
Comment 29•3 years ago
|
||
(In reply to marco.nelissen from comment #28)
No, I didn't enable the preference, because I didn't realize that was necessary in order to actually enable a bugfix from 6 months ago in a build from 2 days ago. Will it ever be enabled by default?
Yes. We would like to enable it by default after finishing Bug 1714238 and Bug 1715387.
Description
•