Skip to content

Commit 874a72a

Browse files
committed
remove now-obsolete comment about autorange/autofill
1 parent 756308f commit 874a72a

File tree

1 file changed

+0
-10
lines changed

1 file changed

+0
-10
lines changed

src/plot_api/plot_api.js

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -2477,16 +2477,6 @@ function applyUIRevisions(data, layout, oldFullData, oldFullLayout) {
24772477
if(preGUIVal === null) preGUIVal = undefined;
24782478
newNP = nestedProperty(layout, key);
24792479
newVal = newNP.get();
2480-
// TODO: This test for undefined is to account for the case where
2481-
// the value was filled in automatically in gd.layout,
2482-
// like axis.range/autorange. In principle though, if the initial
2483-
// plot had a value and the new plot removed that value, we would
2484-
// want the removal to override the GUI edit and generate a new
2485-
// auto value. But that would require figuring out what value was
2486-
// in gd.layout *before* the auto values were filled in, and
2487-
// storing *that* in preGUI... oh well, for now at least I limit
2488-
// this to attributes that get autofilled, which AFAICT among
2489-
// the GUI-editable attributes is just axis.range/autorange.
24902480
if(valsMatch(newVal, preGUIVal)) {
24912481
if(newVal === undefined && key.substr(key.length - 9) === 'autorange') {
24922482
bothInheritAutorange.push(key.substr(0, key.length - 10));

0 commit comments

Comments
 (0)