Branch: refs/heads/master
Home: https://github.com/cytoscape/cytoscape.js
Commit: 608f9a369a59a37c8b482c9b1f9a5240c88396dd
https://github.com/cytoscape/cytoscape.js/commit/608f9a369a59a37c8b482c9b1f9a5240c88396dd
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:
-----------
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 : Bezier edges do not update when sibling edges are removed asynchronously
#2317
--
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.
