Branch: refs/heads/3.3.x
Home: https://github.com/cytoscape/cytoscape.js
Commit: 697947fced9715fccd7122a1cc4a508823784870
https://github.com/cytoscape/cytoscape.js/commit/697947fced9715fccd7122a1cc4a508823784870
Author: Max Franz <[email protected]>
Date: 2019-03-05 (Tue, 05 Mar 2019)
Changed paths:
M src/collection/index.js
M src/collection/style.js
M src/extensions/renderer/base/coord-ele-math/edge-control-points.js
M src/extensions/renderer/base/coord-ele-math/rendered-style.js
M src/style/apply.js
M src/style/properties.js
Log Message:
-----------
Backport 3.3.x: Replace the fix for parallel bundled beziers with a more
robust solution
- The fix needs to be at a lower level: The invalidation of bounds needs to
happen for this case. The rendered style calculation code is too late in the
flow for that.
- This makes the rendered style code simpler again.
- The `ele.remove()` op needs to dirty the bounds for parallel bundled bezier
edges. This should happen only when the removed edge is also bundled so that
other edge types are not negatively affected w.r.t. performance.
- Improve control point check and `isBundledBezier()` check to include more
general `removed()` and `takesUpSpace()` conditions.
- Certain style properties need to specify whether changing the property value
such that a `bounds` is triggered would dirty the bounds of the parallel
bundled bezier edges. For now, this includes the `display` and `curve-style`
properties.
Ref : Backport 3.3.x: Bezier edges do not update when sibling edges are removed
asynchronously #2317 #2324
--
You received this message because you are subscribed to the Google Groups
"cytoscape-cvs" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at https://groups.google.com/group/cytoscape-cvs.
For more options, visit https://groups.google.com/d/optout.
