Bug 331931

Summary: Kickoff's context menu closes popup dialog
Product: [Plasma] plasmashell Reporter: Sebastian Kügler <sebas>
Component: Application Launcher (Kickoff) widgetAssignee: Sebastian Kügler <sebas>
Status: RESOLVED FIXED    
Severity: normal CC: bshah, kde, notmart
Priority: NOR    
Version First Reported In: master   
Target Milestone: 1.0   
Platform: unspecified   
OS: Linux   
Latest Commit: http://commits.kde.org/plasma-desktop/acec3a296f13ba2e972618115e4b6210cd45ff39 Version Fixed/Implemented In:
Sentry Crash Report:
Bug Depends on:    
Bug Blocks: 328586    

Description Sebastian Kügler 2014-03-09 23:04:50 UTC 
When trying to set an item to a favorite in Kickoff, the context menu shows up only sometimes, most of the times it does not. The Kickoff popup dialog closes everytime.

Reproducible: Always

Steps to Reproduce:
1. Click on Kickoff icon
2. search for an application
3. right click on the application item to show the context menu
Actual Results:  
the kickoff popup closes, the context menu either doesn't show up (most of the time), shortly flickers (some of the time) or pops up, but kickoff still closes.

Expected Results:  
Kickoff stays open, context menu stays open until dismissed and works

This seems to be a problem with the transient parent of the ContextMenu, or perhaps an issue in PlasmaCore.Dialog's dismiss-on-focus lost.
Comment 1 Marco Martin 2014-03-10 18:39:57 UTC 
with some contraptions i managed to put the correct transientParent to the menu: menu transientparent == popup dialog.

However it doesn't do anything.
Any ideas?
Comment 2 Marco Martin 2014-04-08 17:04:59 UTC 
Git commit acec3a296f13ba2e972618115e4b6210cd45ff39 by Marco Martin.
Committed on 08/04/2014 at 17:03.
Pushed by mart into branch 'master'.

don't close the menu on submenu open

HACK: is not possible to set a transient parent of a QWindow as a QWidget
because the QWidget's window is not existent until is shown,
at that point it's too late for setting a transient parent

M  +11   -0    applets/kickoff/package/contents/ui/ContextMenu.qml

http://commits.kde.org/plasma-desktop/acec3a296f13ba2e972618115e4b6210cd45ff39