Bug 242173

Summary:	Activating dictionary widget via global shortcut doesn't focus textbox
Product:	[Unmaintained] plasma4	Reporter:	Octavian Voicu <octavian.voicu>
Component:	widget-misc	Assignee:	Plasma Bugs List <plasma-bugs>
Status:	RESOLVED FIXED
Severity:	minor	CC:	asraniel
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Ubuntu
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:

Description Octavian Voicu 2010-06-19 15:17:29 UTC

Version:           unspecified (using Devel) 
OS:                Linux

Textbox is auto-focused only when opening dictionary by clicking its icon. Opening dictionary by keyboard shortcut doesn't let you type a word, unless you click on the textbox. This makes the keyboard shortcut useless.

Reproducible: Always

Steps to Reproduce:
1. Add a dictionary widget on the taskbar.
2. Set a keyboard shortcut for it (let's say Alt+F5).
3. Press Alt+F5.

Actual Results:  
The the dictionary will pop-up, but the textbox won't be focused. There's no way to focus it unless you click on it (or open the dictionary by clicking its icon), so it makes the shortcut totally useless.

Expected Results:  
The textbox should be focused when launching the dictionary from a keyboard shortcut.

Comment 1 Marco Martin 2010-06-20 21:36:41 UTC

SVN commit 1140471 by mart:

delay popup showin in the event queue after activation
fixes focus issues on popups
BUG:242173


 M  +1 -1      popupapplet.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1140471

Comment 2 Octavian Voicu 2010-06-20 23:21:09 UTC

Great, thanks for the quick intervention!

Comment 3 Aaron J. Seigo 2010-12-14 04:15:01 UTC

SVN commit 1206270 by aseigo:

the fix for 242173 in r1140471 broke use cases where the applet is activated and then showPopup(someInterval) is called. the showPopup call would get made, and then it would be overridden at the next event loop with a timeout of 0 due to the QTimer::singleShot in appletActivated. this queues _all_ calls to showPopup using a QBasicTimer, which should address the focus issues a bit more comprehensively for 242173 and not break, e.g., the autohide on the device notifier.
BUG:242173


 M  +38 -16    popupapplet.cpp  
 M  +5 -0      popupapplet.h  
 M  +3 -1      private/popupapplet_p.h  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1206270