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

Tue Jun 7 12:52:43 UTC 2016

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

(Updated Juni 7, 2016, 12:52 nachm.)

Review request for Plasma and KDE Usability.

Changes
-------

* Remove HTML tags and decode HTML entities for copied text

Repository: ktp-desktop-applets

Description
-------

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

Diffs (updated)
-----

  chat/org.kde.ktp-chat/contents/ui/ChatWidget.qml 1485c4c 
  chat/org.kde.ktp-chat/contents/ui/TextDelegate.qml ab686a6 
  chat/qmlplugin/CMakeLists.txt 52e79a0 
  chat/qmlplugin/htmlhelper.h PRE-CREATION 
  chat/qmlplugin/htmlhelper.cpp PRE-CREATION 
  chat/qmlplugin/qmlplugin.cpp 71a6495 

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

Testing (updated)
-------

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.

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/20160607/31001693/attachment.html>