Review Request: Properly strip HTML from assistants' title and action descriptions when converting them to KAction/QMenu.

Ivan Shapovalov intelfx100 at gmail.com
Fri Jan 4 17:48:32 UTC 2013


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

(Updated Jan. 4, 2013, 5:48 p.m.)


Review request for KDevelop.


Changes
-------

Moved the duplicated function to <util/formattinghelpers.h> as that file seemed most suitable.


Description
-------

Fix HTML -> plaintext conversion when converting assistants to KAction/QMenu. This fixes at least one visible case - invoking context menu on a problem with missing declaration assistant.


Diffs (updated)
-----

  interfaces/iassistant.cpp 8d433a3 
  plugins/problemreporter/problemreporterplugin.cpp 27e41ee 
  util/formattinghelpers.h d82d421 
  util/formattinghelpers.cpp b646068 

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


Testing
-------

Apparently works.


Thanks,

Ivan Shapovalov

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20130104/213d6371/attachment.html>


More information about the KDevelop-devel mailing list