Bug 133957 - Click events are not managed in the search engines config dialog
Summary: Click events are not managed in the search engines config dialog
Status: RESOLVED WORKSFORME
Alias: None
Product: konqueror
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Ubuntu Linux
: NOR normal
Target Milestone: ---
Assignee: Konqueror Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-09-12 13:31 UTC by Altmenorg
Modified: 2008-06-01 14:44 UTC (History)
0 users

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 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.