-
Notifications
You must be signed in to change notification settings - Fork 756
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
No Wrapping in arc - Graphics #2754
Comments
@walterbender @pikurasa @meganindya can you please have a look and share your thoughts on this. Thanks |
It makes sense to have wrapping enabled for arc. While we are on the topic, we prob. want to turn off wrapping in the middle of a fill. |
@walterbender sure, it seems reasonable to disable wrapping when drawing for filling. |
Yes. That's correct
…On Mon, Jan 18, 2021, 9:08 PM Kumar Saurabh Raj ***@***.***> wrote:
@walterbender <https://github.com/walterbender> sure, it seems reasonable
to disable wrapping when drawing for filling.
The below screenshot compares the current situation with what is wanted,
is this the kind of functionality we are looking for regarding this?
[image: changes]
<https://user-images.githubusercontent.com/39027928/104978562-186e3700-5a28-11eb-9d1e-564b314a8a1f.jpg>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2754 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA6PXYJ4T2WMD2QJY2L6AXLS2TSSRANCNFSM4WEGABXA>
.
|
@walterbender opened a PR #2765 addressing the wrapping in the middle of fill issue. Please have a look. Thanks |
Forward Block has wrapping (when the turtle goes out of screen bounds) but arc block has no wrapping. This could lead to the turtle getting lost in some situations. It would be better #2750 Proposes a block to enable or disable wrapping, Ideally, we should have wrapping on both forward and arc blocks and the user should be able to enable or disable wrapping.
Is this a possible enhancement or it is intentionally made this way?
Thanks, Please share your feedback and suggestions.
The text was updated successfully, but these errors were encountered: