[bt-devel] Optionsdialog

[Joachim Ansorg]

Hi!

> > I changed the icons.
> > The new KDE2 CVS sources use ListIcon to load list icons, so I changed the code.
> 
> Thanks for clearing that up.
> 
> Can anyone help me on 2 other things:
> 
> First, in the save method for colors, I create a tempColor object and set
> it to each color, such as:
> 
> *tempColor = xxxColorButton->color();
> config->writeEntry("xxx", tempColor->name());
> Can this be combined to avoid using an explicit temp variable.  Would there
> be a benefit to doing it?
 

Try to use something like this:

config->writeEntry("xxx", xxxColorButton->color()->name());

I think this is easier to understand.
 
> Also, in defining the KFontChooser, I want to use the first and last
> arguments of the constructor, and let the rest default.  How would I do
> this?  I have tried various permutations in trying to specify the proper
> values (the booleans are easy, the QStringList seems to be my problem).  I
> am hoping to shrink the size of the overall dialog by only displaying 4 or
> 5 fonts at a time.

It should be possible to use 0 for the QStringList argument.
 
--Joachim

> --
>    Darwin Gregory
> 
>    Help California define marriage the way God does...
>    Support Proposition 22!   Vote YES on Prop 22!
-- 
BibleTime - the bible study program for KDE
http://www.bibletime.de/
info@bibletime.de