> It was > basically just like how I patched it before except this patch is > actually > smaller! I changed this line in pane.js: > > rowStyle+=";position:relative;height:"+rowHeight+"px; > width:"+paneModel.getTotalWidth()+"px;";
That line of code is just after the call to get the row style from the row renderer, which includes setting the colour. So you have got redundant CSS - colour specified twice and a redundant call to the row renderer. Also, because you haven't touched the row renderer's updateDataRowElement which is called when styles are updated in-place, the colours will revert. See the thread I referenced in my other message for more information about how we did it, and the previous problem you had: > So I forced the color of the row to be a given color but the first > rendering of the table uses a black font, but if you click it then > it changes the fonts to the color that I set. So there is another > place where the font color is set but apparently it's being set in > an appearance and not getting over-ridden until the click > happens or something like that. Note that our rowInfo includes the column information, but also additional data in an array (actually the raw data sent from the server). This includes status values which we use to decide on the row colour. Therefore we don't need an additional array or getColors method. Hugh ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ qooxdoo-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
