Differential D4213

Mark non-persistent notifications as transient.
ClosedPublic
Actions

Authored by albertvaka on Jan 19 2017, 11:40 PM.

Details

Reviewers
apol
Group Reviewers
Frameworks
Plasma
Commits
R289:579b75aac8e2: Mark non-persistent notifications as transient.
Summary

On Gnome and other desktops using their extension of the notifications
spec, notifications are persistent unless they set the transient hint.

This tries to make notifications more consistent across desktops.

Test Plan

Manual testing on Gnome.

Diff Detail

Repository
R289 KNotifications
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
albertvaka updated this revision to Diff 10371.Jan 19 2017, 11:40 PM
albertvaka retitled this revision from to Mark non-persistent notifications as transient..
albertvaka updated this object.
albertvaka edited the test plan for this revision. (Show Details)
Restricted Application added a project: Frameworks. · View Herald TranscriptJan 19 2017, 11:40 PM
Restricted Application added a subscriber: Frameworks. · View Herald Transcript
apol added a reviewer: Plasma.Jan 20 2017, 2:33 AM
apol accepted this revision.Jan 21 2017, 10:45 PM
apol added a reviewer: apol.
This revision is now accepted and ready to land.Jan 21 2017, 10:45 PM
Closed by commit R289:579b75aac8e2: Mark non-persistent notifications as transient. (authored by albertvaka). · Explain WhyJan 22 2017, 4:54 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msrc/notifybypopup.cpp (4 lines)

Diff 10423

View Options

src/notifybypopup.cpp

Loading...