Make "Network Disconnected" notification low priority
ClosedPublic

Authored by ngraham on Aug 18 2019, 10:42 PM.

Details

Summary

This is a quintissential low priority notification: nothing bad happens if you miss it,
it's not immediately actionable, and the condition it informs you of is something that
you'll notice anyway. So, let's make it low priority.

Test Plan

For reasons I do not understand, my tests with this diff did not actually make the
"Network Disconnected" notification stay out of the history as intended. I am unsure
whether this is a local issue on my side or if it doesn't work and needs revision.

Diff Detail

Repository
R116 Plasma Network Management Applet
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ngraham created this revision.Aug 18 2019, 10:42 PM
Restricted Application added a project: Plasma. · View Herald TranscriptAug 18 2019, 10:42 PM
Restricted Application added 1 blocking reviewer(s): jgrulich. · View Herald Transcript
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
ngraham requested review of this revision.Aug 18 2019, 10:42 PM
jgrulich accepted this revision.Aug 22 2019, 3:11 PM
This revision is now accepted and ready to land.Aug 22 2019, 3:11 PM
This revision was automatically updated to reflect the committed changes.