From kde-devel  Sat Jan 20 21:25:14 2007
From: Chris Gow <sniffy () rogers ! com>
Date: Sat, 20 Jan 2007 21:25:14 +0000
To: kde-devel
Subject: Re: Issues with kdialog in trunk
Message-Id: <200701201625.14899.sniffy () rogers ! com>
X-MARC-Message: https://marc.info/?l=kde-devel&m=116932834902029

On Saturday 20 January 2007 15:28, Aaron J. Seigo wrote:
> On Saturday 20 January 2007 11:13, Chris Gow wrote:
> > 2. If I change editInformation to not create a KDialog but use the
> > CDInfoDialog class (it is itself a KDialog).
>
> why put a kdialog inside a kdialog instead of just subclassing the dialog
> directly? i just committed a similar change to kscd, btw. i wonder if
> CDInfoDialog was a QWidget or something at one point? hum.... anyways..
>
> > Then the dialog does display
> > all of the fields, but the OK/Cancel buttons are in the middle of the
> > dialog and the separator is near the top.
>
> this was due to a bug in CDInfoDialog. i simply added this to the ctor:
>
> QWidget* w = new QWidget(this);
> setupUi(w);
> setMainWidget(w);
>
> and it shows just fine now =) can you update your copy of kdemultimedia and
> confirm this works for you as well?
Hmmm...I've updated and the fields display but the dialog appears too small.

In tracksimp.cpp I changed this:

KDialog *dialog = new KDialog( this );

to:

CDInfoDialog *dialog = new CDInfoDialog(this);

(and removed all references to base).

If I manually adjust the dialog all appears normal. I guess some sizing is 
missing somewhere?

Btw, just for my education, why is the widget important?

thanks

-- chris
 
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<