Skip to content

Commit

Permalink
Merge pull request #3022 from cytoscape/fix/bezier-display-none
Browse files Browse the repository at this point in the history
Update bundled beziers when setting display: none
  • Loading branch information
maxkfranz committed May 13, 2022
1 parent bcb8804 commit 6f1b58b
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions src/style/apply.js
Original file line number Diff line number Diff line change
Expand Up @@ -821,8 +821,10 @@ styfn.checkBoundsTrigger = function( ele, name, fromValue, toValue ){
// if the prop change makes the bb of pll bezier edges invalid,
// then dirty the pll edge bb cache as well
if( // only for beziers -- so performance of other edges isn't affected
( name === 'curve-style' && (fromValue === 'bezier' || toValue === 'bezier') )
&& prop.triggersBoundsOfParallelBeziers
prop.triggersBoundsOfParallelBeziers
&& ( ( name === 'curve-style' && (fromValue === 'bezier' || toValue === 'bezier') )
|| ( name === 'display' && (fromValue === 'none' || toValue === 'none') )
)
){
ele.parallelEdges().forEach(pllEdge => {
if( pllEdge.isBundledBezier() ){
Expand Down

0 comments on commit 6f1b58b

Please sign in to comment.