Bug 331931 - Kickoff's context menu closes popup dialog
Summary: Kickoff's context menu closes popup dialog
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Application Launcher (Kickoff) (show other bugs)
Version: master
Platform: unspecified Linux
: NOR normal
Target Milestone: 1.0
Assignee: Sebastian Kügler
URL:
Keywords:
Depends on:
Blocks: 328586
  Show dependency treegraph
 
Reported: 2014-03-09 23:04 UTC by Sebastian Kügler
Modified: 2014-04-08 17:04 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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