s434730/RandomSec
3
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fixed minor bug in cell UI that prevented single recon judgment operation from updating the UI.

Browse Source 
Minor optimization: inject main table into the DOM only after it's completely built.

git-svn-id: http://google-refine.googlecode.com/svn/trunk@204 7d457c2a-affb-35e4-300a-418c747d4874
This commit is contained in:
David Huynh 2010-03-05 01:42:21 +00:00
parent 1d6db8fa6e
commit 4d893271ca
2 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/main/webapp/scripts/views/data-table-cell-ui.js
View File

@ -233,7 +233,7 @@ DataTableCellUI.prototype._postProcessOneCell = function(command, params, column
{ columnStatsChanged: columnStatsChanged }, { columnStatsChanged: columnStatsChanged },
{ {
onDone: function(o) { onDone: function(o) {
self._cell = data.cell; self._cell = o.cell;
self._render(); self._render();
} }
} }

8
src/main/webapp/scripts/views/data-table-view.js
View File

@ -85,8 +85,7 @@ DataTableView.prototype.render = function() {
var table = document.createElement("table"); var table = document.createElement("table");
$(table) $(table)
.attr("cellspacing", "0") .attr("cellspacing", "0")
.addClass("data-table") .addClass("data-table");
.appendTo(tableDiv);
var columns = theProject.columnModel.columns; var columns = theProject.columnModel.columns;
var columnGroups = theProject.columnModel.columnGroups; var columnGroups = theProject.columnModel.columnGroups;
@ -258,6 +257,11 @@ DataTableView.prototype.render = function() {
} }
renderRow(tr, r, row, even); renderRow(tr, r, row, even);
} }
/*
* Finally, inject the table into the DOM
*/
$(table).appendTo(tableDiv);
}; };
DataTableView.prototype._showRows = function(start, onDone) { DataTableView.prototype._showRows = function(start, onDone) {