[Kdenlive-devel] Title Outline text

[Marco Gittler]

Am 05.01.2010 um 13:53 schrieb jb:

> On Tuesday 05 January 2010 08:26:24 Marco Gittler wrote:
>> yes this would be nice, if the issue with the color is gone, we are not
>> breaking older titles.
> 
> I just fixed the issue with font color in titlewidget. The problem was that we 
> were using:
> 
> item->setDefaultTextColor(); when there was no outline
> 
> and
> 
> format.setForeground(QBrush(color)); when there was no outline
> 
> So we used 2 different ways to set the font color, which resulted in color 
> being wrongly set.
yes using ->setData(103 would be an option, to handle this in a cleaner way
> 
> We now need to update the loading / saving in titledocument.cpp, and the 
> titler UI needs some love too, we should at least add a label to say what all 
> those sliders mean... not sure if the current layout is optimal...
yes very confusing now. this should be get some work
> 
> Let me know if you want me to do some further work on the titler, and by the 
> way, outline is a great feature!

yes if you want so change these things, this would be very nice.
yes the outline was a feature, is was missing longer ago ;)

regards marco

> 
> regards
> jb
> 
> ------------------------------------------------------------------------------
> This SF.Net email is sponsored by the Verizon Developer Community
> Take advantage of Verizon's best-in-class app development support
> A streamlined, 14 day to market process makes app distribution fast and easy
> Join now and get one step closer to millions of Verizon customers
> http://p.sf.net/sfu/verizon-dev2dev 
> _______________________________________________
> Kdenlive-devel mailing list
> Kdenlive-devel at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/kdenlive-devel

-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 3914 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kdenlive/attachments/20100105/c545ded7/attachment.p7s>