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

Kai Uwe Broulik <[email protected]>
Newsgroups gmane.comp.kde.usability,gmane.comp.kde.devel.plasma
Message-ID <[email protected]>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128089/
-----------------------------------------------------------

(Updated Sept. 27, 2016, 12:50 p.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma and KDE Usability.


Changes
-------

Submitted with commit 8328a1295a7bded16549998411af2446d774dc79 by Kai Uwe Broulik to branch master.


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 
  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
-------

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
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.