-
Notifications
You must be signed in to change notification settings - Fork 177
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
Editor: Fixed right-click in layer panel #11730
Conversation
Essentially just had to make sure that the reorderable only looks for regular left-clicks and ignores other mouse button clicks.
@@ -36,9 +36,10 @@ function useReordering(onPositionChange, numChildren) { | |||
const handleStartReordering = useCallback( | |||
({ position: currentPos, onStartReordering = () => {} }) => | |||
(evt) => { | |||
// Only allow reordering with non-modified click on non-background element. |
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.
The "non-background element" part was left from before the reorderable was generalized out of the layer panel, so this comment does not belong here anymore.
Size Change: +26 B (0%) Total Size: 2.62 MB ℹ️ View Unchanged
|
Plugin builds for 6ac59c6 are ready 🛎️!
|
Should the functionality work for Tested with a mouse and works as expected. |
Oh, that's an excellent question. Then we probably need to detect another key held as well. I'll investigate, thanks! |
Fixed it in 6ac59c6. |
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.
I tested it on my integration branch too - works great 🎉
I think it's safe to say that this doesn't need any additional QA now and can be merged, either way, another round of QA will be conducted on the integration branch on Monday.
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.
Re-tested --- works great.
Essentially just had to make sure that the reorderable only looks for regular left-clicks and ignores other mouse button clicks.
Testing Instructions
This PR can be tested by following these steps:
Checklist
Type: XYZ
label to the PRFixes #11717