Differential D6671

[Notification Item] Enforce PlainText for summary
ClosedPublic
Actions

Authored by broulik on Jul 13 2017, 9:39 AM.

Details

Reviewers
fvogt
davidedmundson
Group Reviewers
Plasma
Commits
R120:ddc530ff14b6: [Notification Item] Enforce PlainText for summary
Summary

According to specification only body text may contain HTML and this keeps apps from screwing up the layout or even loading remote images.

Test Plan

5.8 branch

No longer able to mess up the layout by sending a notification with html tags in title

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.
broulik created this revision.Jul 13 2017, 9:39 AM
Restricted Application added a project: Plasma. · View Herald TranscriptJul 13 2017, 9:39 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson accepted this revision.Jul 13 2017, 9:41 AM
This revision is now accepted and ready to land.Jul 13 2017, 9:41 AM
Closed by commit R120:ddc530ff14b6: [Notification Item] Enforce PlainText for summary (authored by broulik). · Explain WhyJul 13 2017, 9:41 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 16630

View Options

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

Loading...