Re: services runner matching categories

2010-01-10 Thread Aaron J. Seigo
On January 10, 2010, Jacopo De Simoi wrote: > 1) demote exact matches to partial matches to make application names "win"; > this is not perfect, since some popular applications (such as konsole and > dolphin) will quicly climb up the result order; sounds like a good option; they aren't exact match

Re: "Yahoo! Weather" ion dataengine review

2010-01-10 Thread Aaron J. Seigo
On January 10, 2010, Amos Kariuki wrote: > Is there a need to use both kdereview (svn) and reviewboard or should I > just submit a patch for review in reviewboard and ignonre kdereview? no, if it is in playground, move it to kdereview when it's ready. my biggest question before even looking at th

services runner matching categories

2010-01-10 Thread Jacopo De Simoi
Dear plasma-devs After finally upgrading to 4.4 I noticed that the service runner started matching app categories; (the commit indeed happened ~6 months ago) which can be very handy for specific (i.e. single runner, plasma-netbook) queries, but might be imho a bit confusing in krunner common

Re: Review Request: Add support for Opera Bookmarks in the Bookmarks Runner.

2010-01-10 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2539/#review3637 --- Ship it! to me more things supported the better. plus opera bookmarks

"Yahoo! Weather" ion dataengine review

2010-01-10 Thread Amos Kariuki
I've been working on the "Yahoo! Weather" ion dataengine that's currently in Playground (/trunk/playground/base/plasma/ions/yahooweather/) and was thinking of having it moved to Kdereview now that it's feature complete. Are there any instructions on the process to do this or information on how

Review Request: Add support for Opera Bookmarks in the Bookmarks Runner.

2010-01-10 Thread klebezettel
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2539/ --- Review request for Plasma. Summary --- This adds support for Opera Bookmark

[PATCH] Make man:foo, info:foo, etc. work again from KRunner

2010-01-10 Thread Michael Pyne
Hi all, Attached is a patch to fix bug 221371 (man: and info: syntax no longer working) which is caused by a strengthening of the syntax checks done in RunnerContext in kdelibs/plasma. It used to be that near the end of the chain of checks if a context hadn't been defined that a KUrl would be c