Bug 133957

Summary: Click events are not managed in the search engines config dialog
Product: [Applications] konqueror Reporter: Altmenorg <anthony.mercatante>
Component: generalAssignee: Konqueror Developers <konq-bugs>
Status: RESOLVED WORKSFORME    
Severity: normal    
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Ubuntu   
OS: Linux   
Latest Commit: Version Fixed In:

Description Altmenorg 2006-09-12 13:31:53 UTC
Version:            (using KDE KDE 3.5.4)
Installed from:    Ubuntu Packages
OS:                Linux

When configuring the search engines, if you click to select one to have it in the selecbox, "Apply" button doesn't activate. If you then click "OK" your settings are not set...
There are several workarrounds to this.

First one is to select items via the space bar.
The second one is to perform another task that is verified, like for example, after selecting your engines, change and back the default one. Then you can apply the changes.

Here is a patch resolving this issue by simply checking the click event the same way than spacePressed:

diff -Nur kdebase-3.5.4/kcontrol/ebrowsing/plugins/ikws/ikwsopts.cpp kdebase-3.5.4.new/kcontrol/ebrowsing/plugins/ikws/ikwsopts.cpp
--- kdebase-3.5.4/kcontrol/ebrowsing/plugins/ikws/ikwsopts.cpp	2006-01-19 18:00:46.000000000 +0100
+++ kdebase-3.5.4.new/kcontrol/ebrowsing/plugins/ikws/ikwsopts.cpp	2006-09-12 12:05:56.000000000 +0200
@@ -154,6 +154,9 @@
            this, SLOT(checkFavoritesChanged()));
     connect(m_dlg->lvSearchProviders, SIGNAL(spacePressed(QListViewItem *)),
            this, SLOT(checkFavoritesChanged()));
+    connect(m_dlg->lvSearchProviders, SIGNAL(clicked(QListViewItem *)),
+           this, SLOT(checkFavoritesChanged()));
+
 
     connect(m_dlg->cmbDefaultEngine, SIGNAL(activated(const QString &)), this,
             SLOT(configChanged()));

Tested here on kubuntu edgy using 3.5.4, this patch resolves the issue.
Comment 1 FiNeX 2008-06-01 14:44:45 UTC
On 3.5.9 you've to check the checkbox.