Skip to content

Commit 897feb1

Browse files
authored
Fix JS bug in Safari (#306)
Patch for an issue that caused major errors when changing pipeline and hovering nodes in Safari
1 parent 72b2032 commit 897feb1

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

src/components/flowchart/draw.js

+5-1
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,11 @@ export const drawNodes = function(changed) {
126126
const updateNodes = this.el.nodes;
127127
const enterNodes = this.el.nodes.enter().append('g');
128128
const exitNodes = this.el.nodes.exit();
129-
const allNodes = this.el.nodes.merge(enterNodes).merge(exitNodes);
129+
// allNodes includes a further filter to avoid undefined data on Safari
130+
const allNodes = this.el.nodes
131+
.merge(enterNodes)
132+
.merge(exitNodes)
133+
.filter(node => Boolean(node));
130134

131135
if (changed('nodes')) {
132136
enterNodes

0 commit comments

Comments
 (0)