Bug 136065

Summary: focus search field initially
Product: [Applications] systemsettings Reporter: Stefan Monov <logixoul>
Component: generalAssignee: Unassigned bugs mailing-list <unassigned-bugs>
Status: RESOLVED FIXED    
Severity: wishlist CC: finex
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed In:

Description Stefan Monov 2006-10-21 02:42:45 UTC
Version:           3.5.5 "release 19.1"  (using KDE 3.5.5 "release 19.1" , openSUSE )
Compiler:          Target: i586-suse-linux
OS:                Linux (i686) release 2.6.16.21-0.25-default

How to reproduce: run kcontrol.
Observed result: the search field isn't focused.
Expected result: it is.
Comment 1 Stefan Monov 2007-09-12 10:37:28 UTC
same in systemsettings in kde4 beta2
Comment 2 Stefan Monov 2008-05-24 22:18:53 UTC
And here's a patch that does not work :)

Index: workspace/systemsettings/mainwindow.cpp                                                    
===================================================================      
--- workspace/systemsettings/mainwindow.cpp      (revision 811628)
+++ workspace/systemsettings/mainwindow.cpp      (working copy)
@@ -236,6 +236,7 @@
        search->setObjectName(QLatin1String("search"));
     search->setClearButtonShown( true );
     search->setFocusPolicy( Qt::StrongFocus );
+       search->setFocus();
        searchLabel->setBuddy( search );
     // Is that needed? I thought that's what a buddy is for?
        connect(searchText, SIGNAL(triggered()), search, SLOT(setFocus()));
Comment 3 FiNeX 2008-12-31 14:26:27 UTC
This patch works:

Index: mainwindow.cpp
===================================================================
--- mainwindow.cpp      (revisione 903695)
+++ mainwindow.cpp      (copia locale)
@@ -71,7 +71,8 @@
     // We hide the menubar. So ensure the toolbar is always visible because you cannot get it back
     setupGUI(Save|Create,QString());
     menuBar()->hide();
-
+    search->setFocus();
+
     connect(moduleTabs, SIGNAL(currentChanged(int)), SLOT(widgetChange()));
 }




But, should we really apply it?
Comment 4 FiNeX 2008-12-31 14:38:33 UTC
SVN commit 903781 by finex:

Set the focus to the filter bar. (before the focus was on the tab bar).


BUG: 136065



 M  +2 -0      mainwindow.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=903781
Comment 5 Stefan Monov 2008-12-31 15:07:57 UTC
thx!