Differential D899

[notifications] Make the text selectable in the popup too
ClosedPublic
Actions

Authored by mck182 on Feb 4 2016, 4:28 PM.

Details

Reviewers
mart
Group Reviewers
Plasma
Commits
R871:4819282d329f: [notifications] Make the text selectable in the popup too
R120:4819282d329f: [notifications] Make the text selectable in the popup too
Summary

This simplifies the component by removing the Loader-property combo and
leaves just a read-only TextEdit

BUG: 358981

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
mck182 updated this revision to Diff 2190.Feb 4 2016, 4:28 PM
mck182 retitled this revision from to [notifications] Make the text selectable in the popup too.
mck182 updated this object.
mck182 edited the test plan for this revision. (Show Details)
Restricted Application added a project: Plasma. · View Herald TranscriptFeb 4 2016, 4:28 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
mck182 updated this revision to Diff 2192.Feb 4 2016, 4:47 PM

Keep the MouseArea visible always as it pushes the action buttons to the side

mart accepted this revision.Feb 4 2016, 4:56 PM
mart added a reviewer: mart.
This revision is now accepted and ready to land.Feb 4 2016, 4:56 PM
Closed by commit R120:4819282d329f: [notifications] Make the text selectable in the popup too (authored by mck182). · Explain WhyFeb 4 2016, 5:12 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 2193

View Options

applets/notifications/package/contents/ui/NotificationDelegate.qml

Loading...
View Options

applets/notifications/package/contents/ui/NotificationItem.qml

Loading...
View Options

applets/notifications/package/contents/ui/NotificationPopup.qml

Loading...