-
Notifications
You must be signed in to change notification settings - Fork 660
The code is unreasonable #727
Copy link
Copy link
Open
Description
https://github.com/OSBI/saiku/blob/development/saiku-ui/js/saiku/views/TabSet.js
close_all: function() {
for (var i = 0, len = this._tabs.length; i < len; i++) {
var otherTab = this._tabs[i];
otherTab.remove();
}
}should be
close_all: function() {
for (var len = this._tabs.length, i = len - 1; i >= 0; i--) {
var otherTab = this._tabs[i];
otherTab.remove();
}
}Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels