[PATCH]: Kicker find as you type

[Aaron J. Seigo]

On Sunday 01 May 2005 02:08, Dominik Seichter wrote:
> To find the application, I simply search the name, comment and executable
> name of all services.
>
> A screenshot can be found here:
> http://krename.sourceforge.net/screenshots/kicker_01.png

nice ... i believe there is a wishlist request for such an item on b.k.o... at 
first glance, something seems ... hrm.. amis.. i'm not completely sure what 
it is right now, but it seems to be in the wrong place.

<think outloud mode>
almost like i expect it to be in the _main_ menu, not a submenu. perhaps up in 
the "recent apps" area? hrm.

or perhaps in the Applications section, and have it whittle down the options 
so the user can learn where things are as well? (not sure how i feel about 
that, actually)

and it seems to have some overlap with the exec applet as well. as in: this 
functionality would be nice for the exec applet =)

> The patch against HEAD can be found under this URL:
> http://krename.sourceforge.net/data/kicker/kicker-find.patch
>
> To be able to compile with the patch applied, you have to add the files
> http://krename.sourceforge.net/data/kicker/find_mnu.h and
> http://krename.sourceforge.net/data/kicker/find_mnu.cpp to
> kdebase/kicker/ui/ .
>
> Now I would like to know, how this feature can be further improved and
> wether I should commit it to CVS or not.

to be included in CVS it must follow the coding guidelines found in the 
HACKING file.

second, if it is a kpanelmenu subclass, it should be in kicker/menuext (yes, a 
bad name for that directory =) and be made into a loadable module. that way 
it will also appear in the Special Buttons menus and won't require any more 
of those ugly menu-specific hacks in k_menu.cpp nor will it add more clutter 
to kickerSettings.

this means, however, either ditching it updating the Recent Apps list or 
(preferably) abstracting that out to somewhere in libkickermain so everything 
can use it.

and this way, we can put this item in a playground to start with until we 
figure out exactly what to do with it ... the menuext's we have there right 
now really need some sorting out in general.

-- 
Aaron J. Seigo
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20050502/0ac4291e/attachment.sig>