Bug 292503

Summary: Lancelot can only be closed by pressing ESC
Product: [Unmaintained] plasma4 Reporter: Janek Bevendorff <kde>
Component: widget-lancelotAssignee: Ivan Čukić <ivan.cukic>
Status: RESOLVED FIXED    
Severity: normal CC: adaptee
Priority: NOR    
Version: 4.9.0   
Target Milestone: ---   
Platform: Gentoo Packages   
OS: Linux   
Latest Commit: http://commits.kde.org/kdeplasma-addons/8376fb609af92d09b8983451d4220a2e83dc3d8e Version Fixed In:
Sentry Crash Report:

Description Janek Bevendorff 2012-01-26 19:43:01 UTC 
Version:           unspecified (using Devel) 
OS:                Linux

I'm using Lancelot as my application launcher and have assigned the keyboard shortcut Super+Space to it. This shortcut brings up the Lancelot plasmoid and until KDE 4.7.4 I was also able to close it again with this shortcut.
However, since KDE 4.8.0 this doesn't work anymore. The only working shortcut for closing Lancelot is ESC.
This is a little annoying, because sometimes I bring up Lancelot, then decide otherwise and close it again. Using the same shortcut for this is much more convenient, as I don't have to move my hand over the whole keyboard.

Reproducible: Always

Steps to Reproduce:
Assign a keyboard shortcut to Lancelot, open it and try to close it again with the same shortcut.

Actual Results:  
Lancelot only closes on ESC key press.

Expected Results:  
I should also be able to close Lancelot with the same keyboard shortcut I used for opening it.
Comment 1 Ivan Čukić 2012-08-03 22:46:41 UTC 
Git commit a638450103830cc52dfb25d77a40e4dbbd6b07be by Ivan Čukić.
Committed on 04/08/2012 at 00:42.
Pushed by ivan into branch 'master'.

Lancelot is toggled with the keyboard shortcut

M  +8    -1    applets/lancelot/application/LancelotWindow.cpp
M  +5    -0    applets/lancelot/application/LancelotWindow.h
M  +1    -1    applets/lancelot/launcher/LancelotApplet.cpp

http://commits.kde.org/kdeplasma-addons/a638450103830cc52dfb25d77a40e4dbbd6b07be
Comment 2 Ivan Čukić 2012-08-04 08:23:04 UTC 
Git commit 8376fb609af92d09b8983451d4220a2e83dc3d8e by Ivan Čukić.
Committed on 04/08/2012 at 10:15.
Pushed by ivan into branch 'KDE/4.9'.

Backported: Lancelot is toggled with the keyboard shortcut

M  +8    -1    applets/lancelot/application/LancelotWindow.cpp
M  +5    -0    applets/lancelot/application/LancelotWindow.h
M  +1    -1    applets/lancelot/launcher/LancelotApplet.cpp

http://commits.kde.org/kdeplasma-addons/8376fb609af92d09b8983451d4220a2e83dc3d8e