Bug 136065 - focus search field initially
Summary: focus search field initially
Status: RESOLVED FIXED
Alias: None
Product: systemsettings
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR wishlist
Target Milestone: ---
Assignee: Unassigned bugs mailing-list
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-10-21 02:42 UTC by Stefan Monov
Modified: 2008-12-31 15:07 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 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!