Bug 358981 - Unable to select and copy text from notifications panel
Summary: Unable to select and copy text from notifications panel
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Notifications (show other bugs)
Version: master
Platform: Other Linux
: NOR normal
Target Milestone: 1.0
Assignee: Martin Klapetek
URL:
Keywords:
: 367968 (view as bug list)
Depends on:
Blocks:
 
Reported: 2016-02-04 13:16 UTC by kokotokosoko
Modified: 2016-08-29 23:12 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.5.4
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description kokotokosoko 2016-02-04 13:16:37 UTC
This used to work, but no longer does as far as I can tell.

Reproducible: Always

Steps to Reproduce:
1. Generate some persistent (cf bug #358978) notification by whichever means at your disposal.
2. Wait for it to disappear.
3. Click on the notifications icon so as to display the notifications history.
4. Try to select text from a notification.

Actual Results:  
Text cannot be selected.
Context menu does not provide a “copy” function (though without selection, this is substandard).


Expected Results:  
Text should be selectable *and* copyable.


See also bug #358979
Comment 1 kokotokosoko 2016-02-04 14:15:02 UTC
Interestingly, when the notification body does get shown (cf. bug 358979), it is both selectable and copyable. See https://bugs.kde.org/attachment.cgi?id=97010 The word "World" can be selected there.
Comment 2 Martin Klapetek 2016-02-04 15:36:31 UTC
The item in the history popup is selectable. The popup should be too.
That I will fix.

The titles, however, will not be. This is a limitation of the platform
components and I was told there is no plan to make those selectable.

If you want that, you need to file a new bug report against Plasma
components and discuss it there.
Comment 3 kokotokosoko 2016-02-04 15:51:09 UTC
Ok, cheers.

Doesn't clicking on a pop-up causes it to be dismissed? There might be some interaction problems there.
Comment 4 Martin Klapetek 2016-02-04 17:11:52 UTC
Git commit 4819282d329f7462fdc457e63fec58ba4afa37d6 by Martin Klapetek.
Committed on 04/02/2016 at 17:10.
Pushed by mklapetek into branch 'Plasma/5.5'.

[notifications] Make the text selectable in the popup too

This simplifies the component by removing the Loader-property combo and
leaves just a read-only TextEdit
Reviewers: #plasma
Subscribers: plasma-devel
Projects: #plasma

Differential Revision: https://phabricator.kde.org/D899

M  +1    -52   applets/notifications/package/contents/ui/NotificationDelegate.qml
M  +55   -7    applets/notifications/package/contents/ui/NotificationItem.qml
M  +1    -12   applets/notifications/package/contents/ui/NotificationPopup.qml

http://commits.kde.org/plasma-workspace/4819282d329f7462fdc457e63fec58ba4afa37d6
Comment 5 Martin Klapetek 2016-08-29 23:12:48 UTC
*** Bug 367968 has been marked as a duplicate of this bug. ***