Differential D3436

if only title set use as body
AbandonedPublic
Actions

Authored by mart on Nov 21 2016, 5:25 PM.

Details

Reviewers
None
Group Reviewers
Plasma
Summary

some apps are misusing the spec and have a very long
text as title and no body at all.
in this case use that title as body and use the app name as title

BUG:372112

Test Plan

tested with notify-send with or without body.
should this be in 5.8?

Diff Detail

Repository
R120 Plasma Workspace
Branch
phab/longnotiftitles
Lint
No Linters Available
Unit
No Unit Test Coverage
mart updated this revision to Diff 8350.Nov 21 2016, 5:25 PM
mart retitled this revision from to if only title set use as body.
mart updated this object.
mart edited the test plan for this revision. (Show Details)
mart added a reviewer: Plasma.
Restricted Application added a project: Plasma. · View Herald TranscriptNov 21 2016, 5:25 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
mart added a comment.Nov 21 2016, 5:40 PM

uhm... doh! :p

mart abandoned this revision.Nov 21 2016, 5:40 PM

Revision Contents
Changeset List

Diff 8350

View Options

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

Loading...
View Options

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

Loading...