D5794: Copy text from NavigationWidget when selecting

[Kevin Funk]

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


  +1 on the incentive. I had that somewhere on my TODO list...
  
  Also note: Selecting the next, then pressing the middle mouse button already does work. This is not discoverable on other platforms than Linux though.

INLINE COMMENTS

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

That's unexpected behavior.

Better: Use a small "copy" icon, preferrably in the upper right corner, which copies the text to the clipboard when clicked?

REPOSITORY
  R33 KDevPlatform

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

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