Skip to content
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

Add anti-dance heuristics (and feature flags) #1993

Merged
merged 9 commits into from
Nov 24, 2016
Merged

Add anti-dance heuristics (and feature flags) #1993

merged 9 commits into from
Nov 24, 2016

Conversation

davkal
Copy link
Contributor

@davkal davkal commented Nov 7, 2016

This PR introduces new special cases that will not trigger a full re-layout:

  • new 0-degree nodes
  • new nodes that are connected to previous nodes while sharing a rank of other existing nodes

Both heuristics are disabled by default and can be enabled via feature flags, which are introduced here as well (see utils/feature-utils):

In the console you can set localStorage['scope-experimental:layout-dance'] = true to try them out.

Related to #1033

@davkal davkal changed the title [Experimental] Add anti-dance heuristics Add anti-dance heuristics (and feature flags) Nov 15, 2016
Copy link
Contributor

@foot foot left a comment

Choose a reason for hiding this comment

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

lgtm

@@ -197,6 +201,66 @@ class DebugToolbar extends React.Component {
}));
}

setIntermittend() {

This comment was marked as abuse.

}), nodeNames.length),
remove: this._removeNode(),
}));
}

This comment was marked as abuse.

@foot foot assigned davkal and unassigned foot Nov 23, 2016
@davkal davkal merged commit 10602d2 into master Nov 24, 2016
@davkal davkal deleted the dance-tuning branch November 24, 2016 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants