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

Support for ellipses #4

Open
matt-curtis opened this issue Sep 22, 2017 · 3 comments
Open

Support for ellipses #4

matt-curtis opened this issue Sep 22, 2017 · 3 comments

Comments

@matt-curtis
Copy link

matt-curtis commented Sep 22, 2017

Metron is a lovely little library and has saved me a ton of time! Circle works for some cases, but not for ellipses. Seeing this officially supported would be great.

@toineheuvelmans
Copy link
Owner

Hey @matt-curtis, thanks for the compliment, and for your suggestion! Of course, a circle is not an ellipse, but it would be better indeed if ellipses were part of the library. Can you give some suggestions to how you would use such an entity, and what you'd hope to have available to you in terms of properties, functions and such?

@matt-curtis
Copy link
Author

matt-curtis commented Sep 25, 2017

Not very much. The main uses I needed from an Ellipse type was 1. being able to get the area 2. telling whether or not a point falls within the ellipse 3. the bounding rectangle. I ended up quickly building something on top of CGPath(ellipseIn:, transform:).

@davidbjames
Copy link

+1 to having Ellipse(), including Ellipse(in:CGRect).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants