[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-panel-devel
Subject: Re: [PATCH] RunnerManager
From: "Jordi Polo" <mumismo () gmail ! com>
Date: 2008-05-02 12:06:47
Message-ID: a4162420805020506i615812a9q340ec2893dd46888 () mail ! gmail ! com
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
2008/5/2 Aaron J. Seigo <aseigo@kde.org>:
> On Thursday 01 May 2008, Jordi Polo wrote:
> > Done, I've tried to solve the last issues of this mail and already
> commit
> > it.
> > It will be much easier work with it in SVN.
>
> no doubt. i'm so happy that we're on the way with this one.
>
> > Yes, detach on reset, now I hope it is OK (If I believed in deities will
> > pray or something)
>
> i'm building now, will test extensively today ...
>
> what's next on your krunner agenda? personally, i'm going to move all the
> runner related classes into a subdirectory in libplasma to keep them
> together.
Most immediate and easy thing would be create a widget to configure
blacklisting. I will use a KActionSelector (or similar name, don`t have the
sources here to reference). RunnerManager::configWidget to return it? But
I`d like to see the patch from Ryan first as he may be doing something
similar.
About the subdirectory, in fact, each time I rebuild the libs/plasma
directory I feel those few classes are in the wrong place, I mean, most
changes in libplasma will only affect to the krunner GUI and not to the
functionality. Those few classes don`t depend on libplasma or the other way
round.
I would move those classes to a libs/runner and would move runners also
outside plasma directory.
>
> there was a note on the wiki in the API review area about changing the
> name of
> some of the classes so they all started with Runner; i'm still mulling it
> over to decide whether or not i like that, tbh =)
>
> --
> 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 Trolltech
>
> _______________________________________________
> Panel-devel mailing list
> Panel-devel@kde.org
> https://mail.kde.org/mailman/listinfo/panel-devel
>
>
--
Jordi Polo Carres
NLP laboratory - NAIST
http://www.bahasara.org
[Attachment #5 (text/html)]
<br><br><div class="gmail_quote">2008/5/2 Aaron J. Seigo <<a \
href="mailto:aseigo@kde.org">aseigo@kde.org</a>>:<br><blockquote \
class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt \
0pt 0.8ex; padding-left: 1ex;"> <div class="Ih2E3d">On Thursday 01 May 2008, Jordi \
Polo wrote:<br> > Done, I've tried to solve the last issues of this mail and \
already commit<br> > it.<br>
> It will be much easier work with it in SVN.<br>
<br>
</div>no doubt. i'm so happy that we're on the way with this one.<br>
<div class="Ih2E3d"><br>
> Yes, detach on reset, now I hope it is OK (If I believed in deities will<br>
> pray or something)<br>
<br>
</div>i'm building now, will test extensively today ...<br>
<br>
what's next on your krunner agenda? personally, i'm going to move all the<br>
runner related classes into a subdirectory in libplasma to keep them<br>
together.</blockquote><div><br><br>Most immediate and easy thing would be create a \
widget to configure blacklisting. I will use a KActionSelector (or similar name, \
don`t have the sources here to reference). RunnerManager::configWidget to return it? \
But I`d like to see the patch from Ryan first as he may be doing something similar. \
<br> <br>About the subdirectory, in fact, each time I rebuild the libs/plasma \
directory I feel those few classes are in the wrong place, I mean, most changes in \
libplasma will only affect to the krunner GUI and not to the functionality. Those few \
classes don`t depend on libplasma or the other way round. <br> I would move those \
classes to a libs/runner and would move runners also outside plasma directory. \
<br><br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, \
204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"> <br>
<br>
there was a note on the wiki in the API review area about changing the name of<br>
some of the classes so they all started with Runner; i'm still mulling it<br>
over to decide whether or not i like that, tbh =)<br>
<font color="#888888"><br>
--<br>
</font><div><div></div><div class="Wj3C7c">Aaron J. Seigo<br>
humru othro a kohnu se<br>
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA EE75 D6B7 2EB1 A7F1 DB43<br>
<br>
KDE core developer sponsored by Trolltech<br>
</div></div><br>_______________________________________________<br>
Panel-devel mailing list<br>
<a href="mailto:Panel-devel@kde.org">Panel-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/panel-devel" \
target="_blank">https://mail.kde.org/mailman/listinfo/panel-devel</a><br> \
<br></blockquote></div><br><br clear="all"><br>-- <br>Jordi Polo Carres<br>NLP \
laboratory - NAIST<br><a \
href="http://www.bahasara.org">http://www.bahasara.org</a><br>
_______________________________________________
Panel-devel mailing list
Panel-devel@kde.org
https://mail.kde.org/mailman/listinfo/panel-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic