plotly
diff --git a/‎draftlogs/6429_fix.md
Copy file name to clipboard
+1Lines changed: 1 addition & 0 deletions b/‎draftlogs/6429_fix.md
Copy file name to clipboard
+1Lines changed: 1 addition & 0 deletions
diff --git a/‎src/traces/scatter/plot.js
Copy file name to clipboardExpand all lines: src/traces/scatter/plot.js
+1-1Lines changed: 1 addition & 1 deletion b/‎src/traces/scatter/plot.js
Copy file name to clipboardExpand all lines: src/traces/scatter/plot.js
+1-1Lines changed: 1 addition & 1 deletion
diff --git a/‎test/jasmine/tests/scatter_test.js
Copy file name to clipboardExpand all lines: test/jasmine/tests/scatter_test.js
+21Lines changed: 21 additions & 0 deletions b/‎test/jasmine/tests/scatter_test.js
Copy file name to clipboardExpand all lines: test/jasmine/tests/scatter_test.js
+21Lines changed: 21 additions & 0 deletions
@@ -0,0 +1 @@
+ - Fix line redraw (regression introduced in 2.15.0) [[#6429](https://github.com/plotly/plotly.js/pull/6429)]
@@ -248,7 +248,7 @@ function plotOne(gd, idx, plotinfo, cdscatter, cdscatterAll, element, transition
                     revpath = thisrevpath + 'Z' + revpath;
                 }
 
-                if(subTypes.hasLines(trace) && pts.length > 1) {
+                if(subTypes.hasLines(trace)) {
                     var el = d3.select(this);
 
                     // This makes the coloring work correctly:
 
@@ -1035,6 +1035,27 @@ describe('end-to-end scatter tests', function() {
         .then(done, done.fail);
     });
 
+    it('updates line segments on redraw when having null values', function(done) {
+        function checkSegments(exp, msg) {
+            var lineSelection = d3Select(gd).selectAll('.scatterlayer .js-line');
+            expect(lineSelection.size()).toBe(exp, msg);
+        }
+
+        Plotly.newPlot(gd, [{
+            y: [1, null, 2, 3],
+            mode: 'lines+markers'
+        }])
+        .then(function() {
+            checkSegments(2, 'inital');
+
+            return Plotly.relayout(gd, 'xaxis.range', [0, 10]);
+        })
+        .then(function() {
+            checkSegments(2, 'after redraw');
+        })
+        .then(done, done.fail);
+    });
+
     it('correctly autoranges fill tonext traces across multiple subplots', function(done) {
         Plotly.newPlot(gd, [
             {y: [3, 4, 5], fill: 'tonexty'},
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+ - Fix line redraw (regression introduced in 2.15.0) [[#6429](https://github.com/plotly/plotly.js/pull/6429)]`
Original file line number	Diff line number	Diff line change
`@@ -248,7 +248,7 @@ function plotOne(gd, idx, plotinfo, cdscatter, cdscatterAll, element, transition`
`248`	`248`	`revpath = thisrevpath + 'Z' + revpath;`
`249`	`249`	`}`
`250`	`250`
`251`		`- if(subTypes.hasLines(trace) && pts.length > 1) {`
	`251`	`+ if(subTypes.hasLines(trace)) {`
`252`	`252`	`var el = d3.select(this);`
`253`	`253`
`254`	`254`	`// This makes the coloring work correctly:`