[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-panel-devel
Subject: Re: Review request: Container plasma applet
From: Giulio Camuffo <giuliocamuffo () gmail ! com>
Date: 2009-09-20 17:10:47
Message-ID: 9f3bf61b0909201010n56ee719evc788776e9b782b93 () mail ! gmail ! com
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
I know that that functionality belongs in a Containment, but since
containments in containments are not supported i had to use an Applet.
And i know that this applet will never have all the functionalities the
containments have, but this isn't my goal. I wanted to develop a simple
applet to group other applets. I don't plan to implement e.g. the new
feature that loads the applet depending on the mimetype dropped.
I developed this also because I thought to do a plasmoid that was actually a
set of plasmoids inserted in this applet. But if you say that this brings
many bugs and headaches amybe i'll change my mind.
I don't understand however why the systemtray isn't problematic while this
one it is.
I already published it on kde-look, and I'd say people like it, since it is
already 80% good with only 73 downloads, but I thought it would be more
discoverable to the people if it was inside KDE, since, from what I read,
some people need it.
Anyway, with your last sentence you was saying that I could have done a
similar thing working directly on containments?
Giulio
2009/9/20 Aaron J. Seigo <aseigo@kde.org>
> On September 20, 2009, Giulio Camuffo wrote:
> > I've added in kdereview/plasma/applets an applet i made recently. It is,
> as
> > the name suggests, an applet that lets you contain and group other
> > applets.
>
> this functionality belongs in a Containment, not an Applet.
>
> we have two applets right now that can contain other applets: system tray
> and
> system monitor. the latter has had numerous bugs and is, to be frank, an
> implementation mistake. there is nothing that could not have been done much
> easier and with less hacking around stuff if system monitor had not been
> written to embed the various system monitor applets directly.
>
> the design of Plasma is such that the Containment<->Applet relationship is
> quite carefully crafted and solves a lot of issues. ContainerWidget::drop
> which does not support various features found in Containment is a good
> example
> of this.
>
> i think it's a nice candidate for publishing on kde-look.org and you can
> certainly put it into extragear if you'd like, but the concept is
> problematic
> from a design perspective and will cause inconsistencies and other problems
> if
> we ship it with Plasma.
>
> given Containments have complete freedom on how to manage, group, etc.
> Applets, focusing these kinds of efforts there would make a lot more sense.
>
> --
> 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
>
>
[Attachment #5 (text/html)]
I know that that functionality belongs in a Containment, but since containments in \
containments are not supported i had to use an Applet.<br>And i know that this applet \
will never have all the functionalities the containments have, but this isn't my \
goal. I wanted to develop a simple applet to group other applets. I don't plan to \
implement e.g. the new feature that loads the applet depending on the mimetype \
dropped.<br> <br>I developed this also because I thought to do a plasmoid that was \
actually a set of plasmoids inserted in this applet. But if you say that this brings \
many bugs and headaches amybe i'll change my mind.<br>I don't understand \
however why the systemtray isn't problematic while this one it is.<br> <br>I \
already published it on kde-look, and I'd say people like it, since it is already \
80% good with only 73 downloads, but I thought it would be more discoverable to the \
people if it was inside KDE, since, from what I read, some people need it.<br> \
<br>Anyway, with your last sentence you was saying that I could have done a similar \
thing working directly on containments?<br><br><br>Giulio<br><br><div \
class="gmail_quote">2009/9/20 Aaron J. Seigo <span dir="ltr"><<a \
href="mailto:aseigo@kde.org">aseigo@kde.org</a>></span><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="im">On September 20, 2009, Giulio Camuffo \
wrote:<br> > I've added in kdereview/plasma/applets an applet i made recently. \
It is, as<br> > the name suggests, an applet that lets you contain and group \
other<br> > applets.<br>
<br>
</div>this functionality belongs in a Containment, not an Applet.<br>
<br>
we have two applets right now that can contain other applets: system tray and<br>
system monitor. the latter has had numerous bugs and is, to be frank, an<br>
implementation mistake. there is nothing that could not have been done much<br>
easier and with less hacking around stuff if system monitor had not been<br>
written to embed the various system monitor applets directly.<br>
<br>
the design of Plasma is such that the Containment<->Applet relationship is<br>
quite carefully crafted and solves a lot of issues. ContainerWidget::drop<br>
which does not support various features found in Containment is a good example<br>
of this.<br>
<br>
i think it's a nice candidate for publishing on <a href="http://kde-look.org" \
target="_blank">kde-look.org</a> and you can<br> certainly put it into extragear if \
you'd like, but the concept is problematic<br> from a design perspective and will \
cause inconsistencies and other problems if<br> we ship it with Plasma.<br>
<br>
given Containments have complete freedom on how to manage, group, etc.<br>
Applets, focusing these kinds of efforts there would make a lot more sense.<br>
<font color="#888888"><br>
--<br>
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 Qt Development Frameworks<br>
</font><br>_______________________________________________<br>
Plasma-devel mailing list<br>
<a href="mailto:Plasma-devel@kde.org">Plasma-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/plasma-devel" \
target="_blank">https://mail.kde.org/mailman/listinfo/plasma-devel</a><br> \
<br></blockquote></div><br>
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic