-
-
Notifications
You must be signed in to change notification settings - Fork 21.3k
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
Fix various bugs in Animation key right click actions #88445
Conversation
83e0d93
to
e8923cc
Compare
Could you amend the commit message to make it explicit that it's about animation? The current PR title would be a good option. See https://github.com/godotengine/godot/blob/master/CONTRIBUTING.md#be-mindful-of-your-commits for details. |
e8923cc
to
2e42cbe
Compare
@akien-mga of course, sorry for that. Changed commit message to match PR title. |
2e42cbe
to
1f300cd
Compare
rebased on top of 5f05e2b |
1f300cd
to
0413152
Compare
rebased on top of b714563 |
@AThousandShips any chance this PR is reviewed soon? |
No clue, the animation team is requested and will take a look when they can CC @godotengine/animation |
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.
Make sense for me, but I think it is possible to add both to Menu and have two shortcuts such as ctrl + v: without offset
/ ctrl + shift + v: with offset
.
Fix various bugs in Animation key right click actions
06abc86
Thanks! |
When using the features implemented in #87250 from the right-click context menu, they should snap with the current snap steps, same as when inserting a key. They should now work properly in the negative time area, too. Also, there was an error when inserting a key, selecting it, and then performing an undo, that is fixed by these changes.