Bug 166193 - Misspelled applications execute after spelling is corrected
Summary: Misspelled applications execute after spelling is corrected
Status: RESOLVED FIXED
Alias: None
Product: krunner
Classification: Plasma
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
: 167128 (view as bug list)
Depends on:
Blocks:
 
Reported: 2008-07-10 03:46 UTC by Jonathan Thomas
Modified: 2008-12-10 19:22 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
patch to make krunner cancel automatic running after text is changed (355 bytes, patch)
2008-09-04 13:26 UTC, Viesturs Zarins
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Thomas 2008-07-10 03:46:48 UTC
Version:            (using Devel)
Installed from:    Compiled sources
OS:                Linux

Oh the irony if I had misspelled misspelled...
*ahem*
Using the Project Neon KDE Nightly packages, svn 829712.

Steps to reproduce:
-Type Kinqueror in Krunner (Kinky!)
-Hit enter
-Go back and fix your spelling mistake

Expected behavior:
-Konqueror should not run until user presses enter again.

Actual behavior:
-Konqueror starts once your fix the typo.
Comment 1 Chani 2008-07-10 06:01:08 UTC
yeah, that probably shouldn't happen. :)
I'm guessing this has something to do with the "run application even if the user types the name and hits enter before any matches have time to return" feature?
Comment 2 Christophe Marin 2008-07-21 11:56:31 UTC
*** Bug 167128 has been marked as a duplicate of this bug. ***
Comment 3 Pascal Hofmann 2008-09-04 05:23:34 UTC
This also applies to urls. Type "localhost" and press enter -> No "Go to http://localhost" will appear and nothing will happen. Now try to type http:// before localhost - krunner will try to load "http:localhost" as soon as you have typed the colon which leads to the error message "Invalid adress http:localhost". Very annoying behaviour.
Comment 4 Viesturs Zarins 2008-09-04 13:26:11 UTC
Created attachment 27234 [details]
patch to make krunner cancel automatic running after text is changed

Make a very simple fix for this. 
Hope it works. I don't have a full KDE build environment to test it.
Comment 5 Dmitry Suzdalev 2008-10-22 22:04:32 UTC
Dunno if this patch was applied, but krunner in trunk doesn't have this bug - if i correct the misspelled command, it allows me to press enter to execute it :)
Comment 6 Tim Landscheidt 2008-12-10 17:19:11 UTC
When was this patch applied/bug fixed? The krunner in Fedora's kdebase-workspace-4.1.3-6.fc10.i386 still shows this bug so I think "FIXED" is a little euphemism and I'd prefer to go to "REOPENED".
Comment 7 Jonathan Thomas 2008-12-10 19:22:24 UTC
Works in KDE 4.1.80.