Put back call to remove outlines back to original location [1852]

rmoestl · rmoestl · commit d7e64dd949a9 · 2018-08-07T17:43:24.000+02:00
- Reason: to prevent confusion when comparing code to previous versions. - See comment in PR #2824 for more info.
diff --git a/src/plots/cartesian/select.js b/src/plots/cartesian/select.js
@@ -283,11 +283,11 @@ function prepSelect(e, startX, startY, dragOptions, mode) {
                     searchInfo._module.toggleSelected(searchInfo, false);
                 }
 
-                updateSelectedState(gd, searchTraces);
-
                 // clear visual boundaries of selection area if displayed
                 outlines.remove();
 
+                updateSelectedState(gd, searchTraces);
+
                 gd.emit('plotly_deselect', null);
             }
             else {

Original file line number	Diff line number	Diff line change
`@@ -283,11 +283,11 @@ function prepSelect(e, startX, startY, dragOptions, mode) {`
`283`	`283`	`searchInfo._module.toggleSelected(searchInfo, false);`
`284`	`284`	`}`
`285`	`285`
`286`		`- updateSelectedState(gd, searchTraces);`
`287`		`-`
`288`	`286`	`// clear visual boundaries of selection area if displayed`
`289`	`287`	`outlines.remove();`
`290`	`288`
	`289`	`+ updateSelectedState(gd, searchTraces);`
	`290`	`+`
`291`	`291`	`gd.emit('plotly_deselect', null);`
`292`	`292`	`}`
`293`	`293`	`else {`