[Patch] More on KDialog

Olivier Goffart ogoffart at kde.org
Mon Jan 2 12:30:36 GMT 2006


Hi,

After all the discussion, here is a new patch.
It let the KJanusWidget in KDialogBase.

So buttons handling, size handling, and some common stuff are moved to 
KDialog.
And one can add easily a single widget with KDialog::setMainWidget  (the 
common case IMO)

If one want one of the possibility that KJanusWidget offer, KDialogBase is 
still required
(maybe it should be renamed to KJanusDialog)

So if people want to implement stuff based on QAbstractModel , this could be 
done in a dialog that inherit KDialog

Anyway, KDialogBase (KJanusDialog) is not really usefull since it is easy to 
do

KDialog *dialog= new KDialog( this  , "caption");
KJanusWidget *janus=new KJanusWidget( dialog , ... )
dialog->setMainWidget(janus);
janus->addPage( .... )  // like if janus is the KDialogBase

(this is what do KDialogBase internaly in that patch)


Ok to commit ?
(it need still to be polished, but i'd like to commit to not resolve conflict 
everyday)

--
Olivier
-------------- next part --------------
A non-text attachment was scrubbed...
Name: kdialog.tar.gz
Type: application/x-tgz
Size: 23885 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20060102/49518447/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20060102/49518447/attachment.sig>


More information about the kde-core-devel mailing list