Bug 242173 - Activating dictionary widget via global shortcut doesn't focus textbox
Summary: Activating dictionary widget via global shortcut doesn't focus textbox
Status: RESOLVED FIXED
Alias: None
Product: plasma4
Classification: Plasma
Component: widget-misc (show other bugs)
Version: unspecified
Platform: Ubuntu Linux
: NOR minor
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-19 15:17 UTC by Octavian Voicu
Modified: 2010-12-14 04:15 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

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