D7157: Add copy button in AbstractNavigationWidget

Sven Brauch noreply at phabricator.kde.org
Mon Aug 7 08:17:39 UTC 2017


brauch requested changes to this revision.
brauch added a comment.
This revision now requires changes to proceed.


  see inline, otherwise lgtm

INLINE COMMENTS

> abstractnavigationwidget.cpp:114
>  
> +  connect(d->m_browser, &QTextEdit::selectionChanged, this, [=]() {
> +    d->m_browser->copy();

don't use [=], use [this].

REPOSITORY
  R33 KDevPlatform

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

To: patrickelectric, #kdevelop, kfunk, brauch
Cc: kfunk, brauch, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20170807/49bd8be7/attachment.html>


More information about the KDevelop-devel mailing list