Skip to content

Commit 31d5606

Browse files
committed
fixup comments
1 parent c2ecb3a commit 31d5606

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/plot_api/subroutines.js

+2-1
Original file line numberDiff line numberDiff line change
@@ -612,7 +612,7 @@ exports.drawData = function(gd) {
612612
return Plots.previousPromises(gd);
613613
};
614614

615-
// Draw (or redraw) all traces in one go,
615+
// Draw (or redraw) all regl-based traces in one go,
616616
// useful during drag and selection where buffers of targeted traces are updated,
617617
// but all traces need to be redrawn following clearGlCanvases.
618618
//
@@ -624,6 +624,7 @@ exports.drawData = function(gd) {
624624
// non-overlaying, disjoint subplots.
625625
//
626626
// TODO try to include parcoords in here.
627+
// https://github.com/plotly/plotly.js/issues/3069
627628
exports.redrawReglTraces = function(gd) {
628629
var fullLayout = gd._fullLayout;
629630

test/jasmine/tests/splom_test.js

-1
Original file line numberDiff line numberDiff line change
@@ -942,7 +942,6 @@ describe('Test splom interactions:', function() {
942942
});
943943
})
944944
.then(function() {
945-
// TODO should we delete layout.xaxis and layout.yaxis4 here?
946945
_assertAxisTypes('no lower half / no diagonal', {
947946
xaxes: ['linear', 'category', undefined, undefined],
948947
fullXaxes: [null, 'category', 'category', 'log'],

0 commit comments

Comments
 (0)