Bug 211578 - [ regression ] cannot explore list with keyboard
Summary: [ regression ] cannot explore list with keyboard
Status: CLOSED FIXED
Alias: None
Product: krunner
Classification: Plasma
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Unspecified
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords: triaged
Depends on:
Blocks:
 
Reported: 2009-10-23 20:55 UTC by David Palacio
Modified: 2009-12-10 17:50 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Patch to interface.cpp that seems to resolve the above issue (615 bytes, patch)
2009-10-31 17:08 UTC, Jean-Baptiste Rouault
Details

Note You need to log in before you can comment on or make changes to this bug.
Description David Palacio 2009-10-23 20:55:37 UTC
Version:            (using Devel)
Installed from:    Compiled sources

When there are more than two results, press TAB to go to the result list. Then, only the first and last are accessible with the arrow or tab keys (only those are highlighted). Even then, pressing Enter does nothing.
Comment 1 Anselmo L. S. Melo (anselmolsm) 2009-10-24 15:58:59 UTC
Reproducible here, using:
KDE 4.3.72 (KDE 4.3.72 (KDE 4.4 >= 20091015))
Qt 4.6.0 beta1
Comment 2 Jean-Baptiste Rouault 2009-10-31 17:08:38 UTC
Created attachment 37994 [details]
Patch to interface.cpp that seems to resolve the above issue

The above is a small workaround to fix the issue.

The problem is that the scene never receive a QEvent::WindowActivate event because its widget is hidden when KRunner window is shown. I looked at QGraphicsScene code, if a scene never receive the above event, it is considered inactive and it can't take the focus.
Maybe a bug should be filed upstream about this issue ?
Comment 3 Anselmo L. S. Melo (anselmolsm) 2009-11-17 22:36:34 UTC
Patch approved on reviewboard: http://reviewboard.kde.org/r/2190/
SVN commit r105671 by amelo applies the patch proposed.

Thanks to Jean-Baptiste Rouault.

WebSVN link: http://websvn.kde.org/?view=revision&revision=1050671
Comment 4 David Palacio 2009-12-10 14:38:52 UTC
Is this an upstream bug? If it is, does it have a report id?
Comment 5 Anselmo L. S. Melo (anselmolsm) 2009-12-10 17:50:03 UTC
AFAIK it wasn't reported upstream.