Bug 211578

Summary:	[ regression ] cannot explore list with keyboard
Product:	[Plasma] krunner	Reporter:	David Palacio <dpalacio>
Component:	general	Assignee:	Plasma Bugs List <plasma-bugs-null>
Status:	CLOSED FIXED
Severity:	normal	CC:	anselmolsm, jbrouault
Priority:	NOR	Keywords:	triaged
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Compiled Sources
OS:	Unspecified
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	Patch to interface.cpp that seems to resolve the above issue

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.