Review Request: new signal void KMessageWidget::linkActivated()

Dominik Haumann dhaumann at kde.org
Thu Oct 25 14:23:25 UTC 2012


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

Review request for KDE Frameworks and Aurélien Gâteau.


Description
-------

This patch adds a signal void linkActivated(const QString& contents) to KMessageWidget.
The semantics is identical to QLabel::linkActivated(), see: http://doc.qt.digia.com/qt/qlabel.html#linkActivated

Use cases:
- use what's this to show more context help (see diff for kmessagewidgettest)
- open URLs / web pages
- open handbook in KHelpCenter and jump to correct location

Current API: http://api.kde.org/4.9-api/kdelibs-apidocs/kdeui/html/classKMessageWidget.html


Diffs
-----

  kdeui/tests/kmessagewidgettest.cpp be517fc 
  kdeui/widgets/kmessagewidget.h 4169328 
  kdeui/widgets/kmessagewidget.cpp fdda163 

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


Testing
-------

Tested in kmessagewidgettest, see diff.


Thanks,

Dominik Haumann

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20121025/8209a978/attachment.html>


More information about the Kde-frameworks-devel mailing list