-
Notifications
You must be signed in to change notification settings - Fork 206
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
GraphEditor Connection Cutting #5689
Conversation
A-mazing! |
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.
Nice one Murray! A few small comments inline, but hopefully nothing problematic...
Cheers...
John
@@ -726,7 +726,7 @@ void StandardNodeGadget::renderLayer( Layer layer, const Style *style, RenderRea | |||
|
|||
break; | |||
} | |||
case GraphLayer::Overlay : | |||
case GraphLayer::Highlighting : |
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.
This isn't actually sufficient to stop some strikethroughs going on top of the focus gadget - because they are in the same layer, it comes down to which is drawn last. But that's the same as when they were both in the Highlighting layer, so I think that's fine...
fd52dd6
to
808ed53
Compare
Thanks for the input! Pushed a rebase to get |
This gives overlays the opportunity to draw on top of the FocusGadget.
With the FocusGadget moved to render in the Highlighting layer, this change prevents another node's strikethrough from drawing on top of the FocusGadget.
808ed53
to
889c3ea
Compare
Nice one, thanks Murray! I've squashed in all your fixups and merged... |
This was a spare-time couch experiment now tidied up into a PR. Hold
X
and drag to disconnect connections that intersect with the dragged line, orX
and click to disconnect those under the cursor. This functionality should cover the alternate approach proposed in #788.Implemented as a separate gadget to keep the logic out of GraphGadget. For lack of a better idea, I've named this the overly generic "DragEditGadget" as this could potentially be extended to do other things, such as create connections between nodules.
First foray into Gadget land, so suggestions are very welcome...