weather_station applet moved to kdereview

Aaron J. Seigo aseigo at kde.org
Mon Sep 8 19:40:43 CEST 2008


On Monday 08 September 2008, Petri Damstén wrote:
> On Monday 08 September 2008 19:48:34 Anne-Marie Mahfouf wrote:
> > I just tried it. I think you should put the Configure button (like
> > Twitter) so that the user configures it at start.
>
> Yes, that would be more clear than just showing "no cfg" or could it just
> show the settings dialog when there is now config?

setConfigurationRequired(true);

let applet do it for you.

> > You should also send a mail to kde-core-devel and describe the items you
> > moved in kdereview: what it does and where it'll go so core devels can
> > have a look at them.
>
> So good practise would be to send email to kde-core-devel & plasma-devel?

for plasma specific stuff, i don't think kde-core-devel really matters. everyone 
who actually reviews plasma stuff in kdereview is on this list. of course, it 
can't hurt either. *shrug*

> Should there be one email per applet/engine or can they be grouped?

grouped is fine imho.

-- 
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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/plasma-devel/attachments/20080908/382dae73/attachment.sig 


More information about the Plasma-devel mailing list