[System Tray] Don't try to popup empty QMenu
ClosedPublic
Actions

Authored by broulik on May 17 2016, 6:35 PM.

Details

Reviewers

Group Reviewers

Commits

Summary

With KIOSK restrictions it can happen that a context menu would be empty. In this case just a shadow would show up. Do nothing in this case.

Test Plan

AppletInterface already has similar code.
Review pending for ContainmentInterface.

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 retitled this revision from to [System Tray] Don't try to popup empty QMenu.

broulik updated this object.

broulik edited the test plan for this revision. (Show Details)

broulik added a reviewer: Plasma.

broulik set the repository for this revision to R120 Plasma Workspace.

Restricted Application added a subscriber: plasma-devel. · View Herald Transcript

This revision is now accepted and ready to land.May 17 2016, 6:40 PM

This revision was automatically updated to reflect the committed changes.

			Path	Packages
M			applets/systemtray/systemtray.cpp (5 lines)

Diff	ID	Base	Description	Created	Lint	Unit
Base			Base
Diff 1	3850			May 17 2016, 6:35 PM	★	★
Diff 2	3852	1dabe44	rPLASMAWORKSPACEf87e72b980072fa264a1f7013cea397a9c737551	May 17 2016, 7:56 PM	★	★