Skip to content

Allow loopback edges #299

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

MrF3lix
Copy link

@MrF3lix MrF3lix commented Mar 18, 2021

@CLAassistant
Copy link

CLAassistant commented Mar 18, 2021

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

@adriannesoike adriannesoike left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please provide example of expected display. I wasn't able to see the behavior when I checked out to your branch. Thanks!

@@ -664,6 +661,7 @@ class Graph extends React.Component<IGraphProps, IGraphState> {
onCopySelected={this.onCopySelected}
onPasteSelected={this.onPasteSelected}
layoutEngineType={this.state.layoutEngineType}
allowLoopbackEdge={true}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please expose this in example similar to allowMultiselect

graph.edges = [...graph.edges, viewEdge];
this.setState({
graph,
selected: viewEdge,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Think this code needs to be updated (see merge conflicts)

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: Edges with a target node equaling the source node should display as a loopback edge which can be selected
3 participants