Review Request: Better documentation for KConfigDialog::addPage()

Lukas Sommer sommerluk at gmail.com
Sat May 21 12:03:53 BST 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/101409/
-----------------------------------------------------------

(Updated May 21, 2011, 11:03 a.m.)


Review request for kdelibs.


Changes
-------

Updated diff, using the text from Aaron.


Summary
-------

The documentation of KConfigDialog::addPage() is not very detailed about the parameter "pixmapName". This patch adds to the documentation that it is possible to simply pass the KDE icon name instead of the pixmap path.


This addresses bug 273723.
    http://bugs.kde.org/show_bug.cgi?id=273723


Diffs (updated)
-----

  kdeui/dialogs/kconfigdialog.h 873460b 

Diff: http://git.reviewboard.kde.org/r/101409/diff


Testing
-------


Thanks,

Lukas

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20110521/f995a7e2/attachment.htm>


More information about the kde-core-devel mailing list