Skip to content
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

Update path.js #583

Merged
merged 1 commit into from
Dec 22, 2021
Merged

Update path.js #583

merged 1 commit into from
Dec 22, 2021

Conversation

dickinson0718
Copy link
Contributor

Small typo in the comment -- I think that start should be stop in line 77.

I'm searching for a way to to control where zero is in an ellipse. If I declare an ellipse with

this.frontHalf = new Two.Ellipse(0, 0, 0.5, 1);
this.frontHalf.beginning = 0
this.frontHalf.ending = 0.5

Then half of the ellipse should show up, but which half? I was hoping for a command like

https://developer.mozilla.org/en-US/docs/Web/API/CanvasRenderingContext2D/arc

Where I could set a start and end angle. Any insights?

Will

Small typo in the comment -- I think that start should be stop in line 77. 

I'm searching for a way to to control where zero is in an ellipse.  If I declare an ellipse with 

this.frontHalf = new Two.Ellipse(0, 0, 0.5, 1);
this.frontHalf.beginning = 0
this.frontHalf.ending = 0.5

Then half of the ellipse should show up, but which half? I was hoping for a command like

https://developer.mozilla.org/en-US/docs/Web/API/CanvasRenderingContext2D/arc

Where I could set a start and end angle. Any insights?

Will
@dickinson0718
Copy link
Contributor Author

Please ignore my question about the interaction between Ellipse and beginning/ending. I was using an out of date version and since moving to 0.7.13, it works the way I expect! Thanks! Will

@jonobr1
Copy link
Owner

jonobr1 commented Dec 22, 2021

Glad to hear and thanks for contributing!

@jonobr1 jonobr1 merged commit ef4c051 into jonobr1:main Dec 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants