Skip to content

/ servo

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Support SVG circle element #12975

Open
jdm opened this issue Aug 22, 2016 · 7 comments
Open

Support SVG circle element #12975

jdm opened this issue Aug 22, 2016 · 7 comments
Assignees
@stshine
Labels
A-content/dom A-layout/inline

Comments

@jdm
Copy link
Member

@jdm jdm commented Aug 22, 2016

Spec: https://www.w3.org/TR/SVG11/shapes.html#CircleElement

We need to support the cx, cy, r, and fill DOM attributes in order to draw a circle in a benchmark correctly.

Blocks #12973, depends on #12974.
@jdm
Copy link
Member Author

@jdm jdm commented Aug 22, 2016

cc @jrmuizel
@splav
Copy link
Contributor

@splav splav commented Oct 8, 2016

To continue main SVG line I'm taking this one.
@jdm
Copy link
Member Author

@jdm jdm commented Oct 8, 2016

Great!
@jdm jdm added the C-assigned label Oct 8, 2016
@tschneidereit
Copy link
Contributor

@tschneidereit tschneidereit commented Feb 22, 2017

@splav are you still going to work on this? I wasn't intending to work on it myself, but have a contributor who's potentially interested.
@splav
Copy link
Contributor

@splav splav commented Feb 22, 2017

@tschneidereit actially there is a preliminary implementation (https://github.com/splav/servo/tree/SVG_circle%2312975). The thing is, it depends on servo/webrender#402 and should be tested together. And sad to say, right now I've almost no time to spend on servo and webrender at all. So if anyone would like to work on this stuff, please do.
@jdm jdm removed the C-assigned label Feb 22, 2017
@stshine stshine self-assigned this Apr 28, 2017
@stshine stshine added the C-assigned label Apr 28, 2017
@nox
Copy link
Member

@nox nox commented Oct 4, 2017

@stshine Are you still working on this?
@jdm jdm removed the C-assigned label Jun 19, 2018
@jdm
Copy link
Member Author

@jdm jdm commented Jun 19, 2018

An initial attempt exists in #17681 but was abandoned.
@jdm jdm mentioned this issue Jun 19, 2018
Closed
0 of 5 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stshine stshine

Labels
A-content/dom A-layout/inline
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
5 participants
@jdm @tschneidereit @nox @splav @stshine
You can’t perform that action at this time.