diff --git a/chrome/chromeFiles/content/scholar/sidebar.js b/chrome/chromeFiles/content/scholar/sidebar.js index 92f72e097a..8fc9c55932 100644 --- a/chrome/chromeFiles/content/scholar/sidebar.js +++ b/chrome/chromeFiles/content/scholar/sidebar.js @@ -19,8 +19,6 @@ Scholar.TreeView.prototype.setTree = function(treebox) var newRows = Scholar.Items.getTreeRows(); for(var i = 0; i < newRows.length; i++) this._showItem(newRows[i], 0, i+1); //item ref, isContainerOpen, level - - this.rowCount = this._dataItems.length; } Scholar.TreeView.prototype.getCellText = function(row, column) @@ -97,7 +95,6 @@ Scholar.TreeView.prototype.toggleOpenState = function(row) } this._dataItems[row][1] = !this._dataItems[row][1]; //toggle container open value - this.rowCount = this._dataItems.length; this._treebox.rowCountChanged(row+1, count); //tell treebox to repaint these this._treebox.invalidateRow(row); this._treebox.endUpdateBatch(); @@ -112,15 +109,15 @@ Scholar.TreeView.prototype.selectionChanged = function() } else { - removeDynamicRows(); + removeDynamicRows(dynamicBox); document.getElementById('tb-edit').hidden = true; } } -Scholar.TreeView.prototype._showItem = function(item, level, beforeRow) { this._dataItems.splice(beforeRow, 0, [item, false, level]); } +Scholar.TreeView.prototype._showItem = function(item, level, beforeRow) { this._dataItems.splice(beforeRow, 0, [item, false, level]); this.rowCount++; } -Scholar.TreeView.prototype._hideItem = function(row) { this._dataItems.splice(row,1); } +Scholar.TreeView.prototype._hideItem = function(row) { this._dataItems.splice(row,1); this.rowCount--; } Scholar.TreeView.prototype._getItemAtRow = function(row) { return this._dataItems[row][0]; } Scholar.TreeView.prototype.isSorted = function() { return false; } @@ -171,7 +168,6 @@ Scholar.TreeView.prototype.deleteSelection = function() //remove row from tree this._hideItem(rows[i]-i); - this.rowCount--; this._treebox.rowCountChanged(rows[i]-i, -1); } this._treebox.endUpdateBatch(); @@ -185,7 +181,7 @@ Scholar.DragObserver.drop = function(row, orient) { } function viewSelectedItem() { - removeDynamicRows(); + removeDynamicRows(dynamicBox); var thisItem = myTreeView._getItemAtRow(myTreeView.selection.currentIndex); @@ -246,7 +242,7 @@ function editItem(thisItem) document.getElementById('list-pane').hidden = true; document.getElementById('edit-pane').hidden = false; - removeDynamicRows(); + removeDynamicRows(dynamicBox); var fieldNames = getFullFieldList(thisItem); for(var i = 0; i + flex="1" + hidden="true"/> + +