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

Error in how polygons are drawn #178

Closed
imeckler opened this Issue Feb 19, 2015 · 5 comments

Comments

Projects
None yet
3 participants
@imeckler
Contributor

imeckler commented Feb 19, 2015

There is an error in the way polygons are draw. In short, the path defining the polygon is somehow not really being closed, and so linejoins are not rendered correctly. Here's an example showing smooth line joins being rendered jaggedly.

http://share-elm.com/sprout/54e64591e4b09711f39c2b0d

@vilterp

This comment has been minimized.

Show comment
Hide comment
@vilterp

vilterp Feb 21, 2015

Fixed by using cap <- Padded in the above example

http://share-elm.com/sprout/54e90d3be4b09711f39c2c88

vilterp commented Feb 21, 2015

Fixed by using cap <- Padded in the above example

http://share-elm.com/sprout/54e90d3be4b09711f39c2c88

@imeckler

This comment has been minimized.

Show comment
Hide comment
@imeckler

imeckler Feb 21, 2015

Contributor

I wouldn't call that "fixed", it's just a different thing since the corners are sharp rather than smooth.

Contributor

imeckler commented Feb 21, 2015

I wouldn't call that "fixed", it's just a different thing since the corners are sharp rather than smooth.

@vilterp

This comment has been minimized.

Show comment
Hide comment
@vilterp

vilterp Feb 22, 2015

Fair. Seems like it isn't treating the lower right corner as a join, and is basically just drawing it as a path. Not quite sure where to fix this in Graphics.Collage.

vilterp commented Feb 22, 2015

Fair. Seems like it isn't treating the lower right corner as a join, and is basically just drawing it as a path. Not quite sure where to fix this in Graphics.Collage.

@evancz

This comment has been minimized.

Show comment
Hide comment
@evancz

evancz Feb 22, 2015

Member

Is the fix "you can't draw closed polygons with lines"?

Currently the internal representation of Forms does not have any special cases, but it'd be good to have for circles and rectangles probably. I'd like to do a big set of improvements like this that includes supporting arcs, bezier curves, etc. but I think it'll be a while until then unfortunately :/

Member

evancz commented Feb 22, 2015

Is the fix "you can't draw closed polygons with lines"?

Currently the internal representation of Forms does not have any special cases, but it'd be good to have for circles and rectangles probably. I'd like to do a big set of improvements like this that includes supporting arcs, bezier curves, etc. but I think it'll be a while until then unfortunately :/

@evancz

This comment has been minimized.

Show comment
Hide comment
@evancz

evancz May 11, 2016

Member

Sorry this did not get attention til now! The Graphics.* modules now live in evancz/elm-graphics so I am trying to get stuff migrated over there.

I don't really know what action to take on this one though, so unless there is a proposed fix, I'm not sure how valuable it is to keep this open.

Member

evancz commented May 11, 2016

Sorry this did not get attention til now! The Graphics.* modules now live in evancz/elm-graphics so I am trying to get stuff migrated over there.

I don't really know what action to take on this one though, so unless there is a proposed fix, I'm not sure how valuable it is to keep this open.

@evancz evancz closed this May 11, 2016

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