- 
          
 - 
                Notifications
    
You must be signed in to change notification settings  - Fork 972
 
Add selection subtracting for node graph #3290
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
base: master
Are you sure you want to change the base?
Add selection subtracting for node graph #3290
Conversation
| 
           !build  | 
    
          
  | 
    
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.
Works like described, implementation LGTM.
Not sure if we want this shortcut though, for me this feels like a good idea.
Also did not look for conflicts with other key binds.
| HintGroup(vec![ | ||
| HintInfo::mouse(MouseMotion::LmbDrag, "Select Area"), | ||
| HintInfo::keys([Key::Shift], "Extend").prepend_plus(), | ||
| HintInfo::keys([Key::Alt], "Subtract").prepend_plus(), | 
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.
Not sure if subtracting is the right terminology.
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.
It should be, that's how the graphite select tool and the wider ecosystem terms it: https://www.google.com/search?q=subtract%20selection (though I believe Pinta uses the proper set vocabulary which is pretty neat)
2573298    to
    220d630      
    Compare
  
    220d630    to
    e721e25      
    Compare
  
    | 
           !build  | 
    
          
  | 
    
| 
           Just keeping things rebased with master. Thanks for the rebuild 👍  | 
    
Does not close any issue, but I was looking into #2532, a Paper Cut issue that requests parity between the canvas and node graph by adding the lasso select, which has no keybind conflicts. This is my very first PR here, and it was a bit out of my depth.
I wanted to start a bit smaller, and noticed that the node graph could include alt-to-subtract selections without keybind conflicts, because none of the alt keybinds correspond to left clicks on the background!
2025-10-15.16-54-19.mp4
All the other Alt keybinds like Move Duplicate and Preview Node Output still work as expected.