[Notifications] Move "Open With" items into the top group of the menu
ClosedPublic

Authored by ngraham on May 14 2018, 11:25 PM.

Details

Summary

Move the Open With items into the top group of the context menu for notifications with images or files that you can open and see in the filesystem. This makes it consistent with Dolphin, which now has all the Open in/with items grouped together at the top of the menu.

I didn't move the Open With items all the way to the top because my sense is that Open containing folder is the more common action and deserves to be the top item. Also, putting it below the Open With actions would result in an extra ugly separator.

Test Plan

Diff Detail

Repository
R120 Plasma Workspace
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
ngraham created this revision.May 14 2018, 11:25 PM
Restricted Application added a project: Plasma. · View Herald TranscriptMay 14 2018, 11:25 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
ngraham requested review of this revision.May 14 2018, 11:25 PM
ngraham edited the test plan for this revision. (Show Details)May 14 2018, 11:27 PM
ngraham edited the summary of this revision. (Show Details)
broulik accepted this revision.May 15 2018, 7:53 AM

Thanks!

This revision is now accepted and ready to land.May 15 2018, 7:53 AM
ngraham closed this revision.May 15 2018, 12:57 PM