Review Request 128089: [Quick Chat] Add context menu for copying text and links

Kai Uwe Broulik kde at privat.broulik.de
Fri Jun 3 14:26:29 UTC 2016


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

Review request for Plasma and KDE Usability.


Repository: ktp-desktop-applets


Description
-------

This allows to copy links by right clicking them and copying individual text snippets to the clipboard.


Diffs
-----

  chat/org.kde.ktp-chat/contents/ui/ChatWidget.qml 1485c4c 
  chat/org.kde.ktp-chat/contents/ui/TextDelegate.qml ab686a6 

Diff: https://git.reviewboard.kde.org/r/128089/diff/


Testing
-------

Unfortunately QtQuick has no proper text selection and you can only copy one message at a time, not an entire paragraph from a user, although it looks that way because they're grouped. Better than nothing I suppose.

Just noticed the "Copy Text" also copies html tags, ie. <a href=".."> when text contains a link ("Copy Link" really just copies the link)


File Attachments
----------------

Copy link
  https://git.reviewboard.kde.org/media/uploaded/files/2016/06/03/15b0f08e-9a54-46c3-9d87-e817a7dfbef1__Screenshot_20160603_161915.png
Copy text
  https://git.reviewboard.kde.org/media/uploaded/files/2016/06/03/e431324f-de43-4d17-9f65-9f16f5af2f92__Screenshot_20160603_161931.png


Thanks,

Kai Uwe Broulik

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160603/5081939e/attachment.html>


More information about the Plasma-devel mailing list