D12807: Akregator feed detector plugin: Port the context menu plugin

Jonathan Marten noreply at phabricator.kde.org
Fri May 11 08:33:15 BST 2018


marten updated this revision to Diff 33971.
marten marked 6 inline comments as done.
marten added a comment.


  Diff updated in accordance with review comments: coding style, modern C++ features and reducing visibility wherever possible.
  
  David, would you also be able to look at https://phabricator.kde.org/D12616 - KF5 conversion of the other target - so that they can both be committed in sequence?

REPOSITORY
  R226 Konqueror

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D12807?vs=33947&id=33971

REVISION DETAIL
  https://phabricator.kde.org/D12807

AFFECTED FILES
  plugins/akregator/CMakeLists.txt
  plugins/akregator/akregator_konqplugin.desktop
  plugins/akregator/akregatorplugin.cpp
  plugins/akregator/akregatorplugin.h

To: marten, #konqueror, #plasma, dfaure
Cc: dfaure
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180511/d3f5cabe/attachment.htm>


More information about the kfm-devel mailing list