KConfigDialog patch

[Joris Guisson]

OK, this is the same patch but with the default button being updated. It
appears to work properly, when I tested it KTorrent.

I also updated the doxygen comments, and added my name to the copyrights
list.

Joris,

On 8/12/07, Joris Guisson <joris.guisson at gmail.com> wrote:
>
>
>
> On 8/12/07, Aaron J. Seigo <aseigo at kde.org> wrote:
> >
> > On Saturday 11 August 2007, Joris Guisson wrote:
> > > It seems to be in the API for quite some time now, so it's a bit
> > strange I
> > > never discovered this before.
> >
> > it needs a techbase tutorial =)
> >
> > > This all then leads to the attached patch. Can this patch be included
> > in
> > > kdelibs ?
> >
> > it seems that the (some of the?) check(s) currently done in showEvent()
> > should
> > probably also be done when addPage is called? that seems to be the
> > reason why
> > currently pages aren't allowed to be added, from what i can discern from
> > the
> > code?
>
>
> I guess only the default button would have to be updated, the
> updateWidgets stuff has all been done for the older pages, and for the new
> page, my change in setupWidget would take care of that. The apply button
> doesn't need to be changed, because nothing has changed in the new page.
>
>
> Joris,
>
> --
> > Aaron J. Seigo
> > humru othro a kohnu se
> > GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43
> >
> > KDE core developer sponsored by Trolltech
> >
> >
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20070813/676309d5/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: kconfigdialog.patch
Type: text/x-patch
Size: 4199 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20070813/676309d5/attachment.bin>