Hi Dietrich,

thank you for your patch. Some comments:

It may be better to add a new method to Font.js to support a 
generateStyle() method

In _renderFont you then could use something like this.__fontStyle = 
value ? value.generateStyle() : "";

Then you can add the generates style to the htmlArr.push in the other 
methods.

And please be sure to also support all the font features e.g. bold, 
italic, underlined etc.

Sebastian



Dietrich Streifert schrieb:
> Hi David,
> 
> I've created a patch against trunk which allows the styling of the table 
> font through appearance or through font setting of the default row renderer.
> 
> If you want to try this take a look at attachment
> 
>     http://bugzilla.qooxdoo.org/attachment.cgi?id=192
> 
> of bug
> 
>     http://bugzilla.qooxdoo.org/show_bug.cgi?id=493
> 
> Sebastian: any time to review my patch? Is this the right way?
> 
> 
> dperez schrieb:
>> Hi,
>>
>> I've discovered that for the Ext theme, the font of the textfield of a
>> ComboBoxEx and the font of the list isn't the same.
>>
>> Does anyone know if there is an API for changing the font of a table?
>> Tables are a little different than other widgets, HTML is generated
>> directly.
>>
>> Cheers,
>> David
>>   
> 


-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
qooxdoo-devel mailing list
qooxdoo-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel

Reply via email to