Re: Character-Runner

2010-04-12 Thread Burkhard Lück
Am Freitag, 9. April 2010 00:57:10 schrieb Aaron J. Seigo:
 On April 8, 2010, Anton Kreuzkamp wrote:
  I would appreciate it if you could review it and possibly integrate it
  into Trunk. Thanks.
 
 looks good. two comments on it:
 
 * context.query() is already trim()'d, so you don't have to re-trim() them.
 
 * in reloadConfiguration, the runner should create one or more QuerySyntax
 objects and register them with addSyntax or setSyntaxes.
 
 other than that, if you'd like to maintain this in KDE's svn, that'd be
  cool. you'll need an svn account[1] and commit to maintaining the plugin.
  it would go into kdeplasma-addons/runners.
 
It arrived there now, but the runner is broken i18n-wise.

Messages are extracted to plasma_runner_charrunner, but via the first argument 
in the K_EXPORT_PLASMA_RUNNER macro it tries to load the non existing catalog 
plasma_runner_CharacterRunner.

How to fix that bug, renaming the catalog or the arg in K_EXPORT_PLASMA_RUNNER 
macro?

-- 
Burkhard Lück
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Character-Runner

2010-04-12 Thread Aaron J. Seigo
On April 12, 2010, Burkhard Lück wrote:
 How to fix that bug, renaming the catalog or the arg in
 K_EXPORT_PLASMA_RUNNER macro?

i changed the name of the catalog.

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Qt Development Frameworks
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Character-Runner

2010-04-08 Thread Anton Kreuzkamp




A few days ago I wrote a KRunner-plugin for creating special characters from its hexadecimal-code. (e.g. #3b1 for ).
It also has the possibility to configure aliases (e.g. #a instead of #3b1):
http://kde-look.org/content/show.php/Character+Runner?content=122629

I would appreciate it if  you could review it and possibly integrate it into Trunk.
Thanks.

With best regards, Anton
GRATIS fr alle WEB.DE-Nutzer: Die maxdome Movie-FLAT!Jetzt freischalten unter http://movieflat.web.de

___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


Re: Character-Runner

2010-04-08 Thread Aaron J. Seigo
On April 8, 2010, Anton Kreuzkamp wrote:
 I would appreciate it if you could review it and possibly integrate it into
 Trunk. Thanks.

looks good. two comments on it:

* context.query() is already trim()'d, so you don't have to re-trim() them.

* in reloadConfiguration, the runner should create one or more QuerySyntax 
objects and register them with addSyntax or setSyntaxes.

other than that, if you'd like to maintain this in KDE's svn, that'd be cool. 
you'll need an svn account[1] and commit to maintaining the plugin. it would 
go into kdeplasma-addons/runners.

[1] http://techbase.kde.org/Contribute/Get_a_SVN_Account

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Qt Development Frameworks
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel