-
-
Notifications
You must be signed in to change notification settings - Fork 21.4k
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
Clarify Tween.set_parallel()
#79758
Clarify Tween.set_parallel()
#79758
Conversation
I fell under the same "trap" right now, so it would be nice to have this one looked at. Although I kind of have to agree, the behavior is a bit unorthodox. |
Yeah, I don't get it. It makes everything after and one before to run in parallel? I can see the logic behind it now, but without knowing that this is the case I'd be very surprised by this behavior. |
It feels buggy, but would be too late to somehow change now. I wouldn't hope others to completely disregard parallel tweens for this undocumented behavior |
How do we feel about this? I still believe this behavior is unorthodox, but it's better to document it than not doing anything. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're right, Micky from 3 weeks ago: it is better to document it than not doing anything.
Although the current behavior is not exactly pretty.
Thanks! |
Cherry-picked for 4.2.2. |
Addresses #74608 (comment)