Hi there,

in the current version[1] qx.ui.table.Table#updateContent() does only update
the current selected row.
This prevents for example the localization of all table-content through a
cellrenderer.

The same problem was already "fixed" by Derrell here:
http://qooxdoo.678.n2.nabble.com/cellrenderer-Boolean-change-the-icons-in-runtime-bug-tp5069023p5069023.html

I think the fix is to change the table's updateContent method just like in the
attached patches [calling 'qx.ui.table.pane.Pane#updateContent()' with the
completeUpdate parameter set to 'true'].

AFAIK there is currently no bug in bugzilla about this --shall I create one?

/Peter

[1] qoodxdoo 1.1 ...and trunk as well
Index: Table.js
===================================================================
--- Table.js    (Revision 22477)
+++ Table.js    (Arbeitskopie)
@@ -1708,7 +1708,7 @@
     updateContent : function() {
       var scrollerArr = this._getPaneScrollerArr();
       for (var i=0; i<scrollerArr.length; i++) {
-        scrollerArr[i].getTablePane().updateContent();
+        scrollerArr[i].getTablePane().updateContent(true);
       }
     },
 
Index: Table.js
===================================================================
--- Table.js    (Revision 22477)
+++ Table.js    (Arbeitskopie)
@@ -1720,7 +1720,7 @@
     updateContent : function() {
       var scrollerArr = this._getPaneScrollerArr();
       for (var i=0; i<scrollerArr.length; i++) {
-        scrollerArr[i].getTablePane().updateContent();
+        scrollerArr[i].getTablePane().updateContent(true);
       }
     },
 
------------------------------------------------------------------------------
ThinkGeek and WIRED's GeekDad team up for the Ultimate 
GeekDad Father's Day Giveaway. ONE MASSIVE PRIZE to the 
lucky parental unit.  See the prize list and enter to win: 
http://p.sf.net/sfu/thinkgeek-promo
_______________________________________________
qooxdoo-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel

Reply via email to