[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-devel
Subject:    Re: Issues with kdialog in trunk
From:       Chris Gow <sniffy () rogers ! com>
Date:       2007-01-20 21:25:14
Message-ID: 200701201625.14899.sniffy () rogers ! com
[Download RAW message or body]

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 <<
[prev in list] [next in list] [prev in thread] [next in thread]