[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    Re: Changes to Plasma Weather - Git revision 5a09cd815edd7723800c0ba43a8be9f67d28e011
From:       Luís_Gabriel <lampih () gmail ! com>
Date:       2012-09-11 11:30:50
Message-ID: CANV84AMYC+yFXftbhyLBXcmAZ4PMVJiFsZjUW+fCu3QDu=gv0A () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


You can see some screenshots here:
https://git.reviewboard.kde.org/r/106225/
There was no changes in the UI, it was just rewritten in QML.

Cheers,
--
Lu=EDs Gabriel


On Tue, Sep 11, 2012 at 8:15 AM, Shawn Starr <shawn.starr@rogers.com> wrote=
:

> On Tuesday, September 11, 2012 07:58:16 AM Lu=EDs Gabriel wrote:
> > Hi  Shawn,
> >
> > I don't see the point of revert the changes.
> > Sorry about the misunderstanding but as you see in the previous replies=
,
> > not everyone were aware of your plans to work on this plasmoid.
> > Feel free to work on a new UI if you want, but please do it on top of t=
he
> > current master.
> >
> > Cheers,
> > --
> > Lu=EDs Gabriel
> >
>
> Hello,
>
> I would rather not have to fork my own project so I won't be doing that. =
I
> need to see screenshot of the changes being made since removing
> information is
> not part of my plans for this applet, cleaning up graphic elements is
> however.
>
> Thanks,
> Shawn.
>
> > On Tue, Sep 11, 2012 at 5:55 AM, Marco Martin <notmart@gmail.com> wrote=
:
> > > On Tuesday 11 September 2012, Shawn Starr wrote:
> > > > Hi Lu=EDs,
> > > >
> > > > I appreciate you making changes to the code, but it would have been
> nice
> > >
> > > to
> > >
> > > > been told about this since I am about to replace this with QML
> porting.
> > > >
> > > > Please discuss with me what changes you would like to make as I am =
in
> > > > the
> > > > process of porting to QML for the applet. My goals for '2.0' are
> > >
> > > different
> > >
> > > > from what you're doing.
> > > >
> > > > I would like to revert this out of trunk or discuss this on IRC
> please.
> > >
> > > ok, let me be clear about this.
> > > there won't be any revert.
> > >
> > > all the changes were discussed and toroughly reviewed. It's unfortuna=
te
> > > that
> > > you didn't participate in it and yes, it's also unfortunate that we
> didn't
> > > contact you.
> > >
> > > when the rewrite started, that applet was effectively abandoned and
> > > bitrotting
> > > since at least two years, as unfortunately are many other applets in
> > > kdeplasma-addons, and this requires an action (other addons plasmoids
> will
> > > soon share the same fate)
> > >
> > > about the 2.0 you have in mind, the current design well fits in the
> global
> > > style (simple display of information that cuts useless graphic
> elements to
> > > the
> > > minimum) that is the style the default plasmoids should have.
> > >
> > > However, if you want something different, you are more than welcome a=
nd
> > > encouraged to work on another plasmoid (and then using the same revie=
w
> > > process
> > > that was used in this case)
> > >
> > > Cheers,
> > > Marco Martin
> > > _______________________________________________
> > > Plasma-devel mailing list
> > > Plasma-devel@kde.org
> > > https://mail.kde.org/mailman/listinfo/plasma-devel
>

[Attachment #5 (text/html)]

You can see some screenshots here: 
<a href="https://git.reviewboard.kde.org/r/106225/">https://git.reviewboard.kde.org/r/106225/</a> \
<div>There was no changes in the UI, it was just rewritten in \
QML.<br><div><br></div><div>Cheers,<br clear="all">--<br>Luís Gabriel<br>


<br><br><div class="gmail_quote">On Tue, Sep 11, 2012 at 8:15 AM, Shawn Starr <span \
dir="ltr">&lt;<a href="mailto:shawn.starr@rogers.com" \
target="_blank">shawn.starr@rogers.com</a>&gt;</span> wrote:<br><blockquote \
class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex">

<div class="im">On Tuesday, September 11, 2012 07:58:16 AM Luís Gabriel wrote:<br>
&gt; Hi  Shawn,<br>
&gt;<br>
&gt; I don&#39;t see the point of revert the changes.<br>
&gt; Sorry about the misunderstanding but as you see in the previous replies,<br>
&gt; not everyone were aware of your plans to work on this plasmoid.<br>
&gt; Feel free to work on a new UI if you want, but please do it on top of the<br>
&gt; current master.<br>
&gt;<br>
&gt; Cheers,<br>
&gt; --<br>
&gt; Luís Gabriel<br>
&gt;<br>
<br>
</div>Hello,<br>
<br>
I would rather not have to fork my own project so I won&#39;t be doing that. I<br>
need to see screenshot of the changes being made since removing information is<br>
not part of my plans for this applet, cleaning up graphic elements is however.<br>
<br>
Thanks,<br>
Shawn.<br>
<div class="HOEnZb"><div class="h5"><br>
&gt; On Tue, Sep 11, 2012 at 5:55 AM, Marco Martin &lt;<a \
href="mailto:notmart@gmail.com">notmart@gmail.com</a>&gt; wrote:<br> &gt; &gt; On \
Tuesday 11 September 2012, Shawn Starr wrote:<br> &gt; &gt; &gt; Hi Luís,<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; I appreciate you making changes to the code, but it would have been \
nice<br> &gt; &gt;<br>
&gt; &gt; to<br>
&gt; &gt;<br>
&gt; &gt; &gt; been told about this since I am about to replace this with QML \
porting.<br> &gt; &gt; &gt;<br>
&gt; &gt; &gt; Please discuss with me what changes you would like to make as I am \
in<br> &gt; &gt; &gt; the<br>
&gt; &gt; &gt; process of porting to QML for the applet. My goals for &#39;2.0&#39; \
are<br> &gt; &gt;<br>
&gt; &gt; different<br>
&gt; &gt;<br>
&gt; &gt; &gt; from what you&#39;re doing.<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; I would like to revert this out of trunk or discuss this on IRC \
please.<br> &gt; &gt;<br>
&gt; &gt; ok, let me be clear about this.<br>
&gt; &gt; there won&#39;t be any revert.<br>
&gt; &gt;<br>
&gt; &gt; all the changes were discussed and toroughly reviewed. It&#39;s \
unfortunate<br> &gt; &gt; that<br>
&gt; &gt; you didn&#39;t participate in it and yes, it&#39;s also unfortunate that we \
didn&#39;t<br> &gt; &gt; contact you.<br>
&gt; &gt;<br>
&gt; &gt; when the rewrite started, that applet was effectively abandoned and<br>
&gt; &gt; bitrotting<br>
&gt; &gt; since at least two years, as unfortunately are many other applets in<br>
&gt; &gt; kdeplasma-addons, and this requires an action (other addons plasmoids \
will<br> &gt; &gt; soon share the same fate)<br>
&gt; &gt;<br>
&gt; &gt; about the 2.0 you have in mind, the current design well fits in the \
global<br> &gt; &gt; style (simple display of information that cuts useless graphic \
elements to<br> &gt; &gt; the<br>
&gt; &gt; minimum) that is the style the default plasmoids should have.<br>
&gt; &gt;<br>
&gt; &gt; However, if you want something different, you are more than welcome and<br>
&gt; &gt; encouraged to work on another plasmoid (and then using the same review<br>
&gt; &gt; process<br>
&gt; &gt; that was used in this case)<br>
&gt; &gt;<br>
&gt; &gt; Cheers,<br>
&gt; &gt; Marco Martin<br>
&gt; &gt; _______________________________________________<br>
&gt; &gt; Plasma-devel mailing list<br>
&gt; &gt; <a href="mailto:Plasma-devel@kde.org">Plasma-devel@kde.org</a><br>
&gt; &gt; <a href="https://mail.kde.org/mailman/listinfo/plasma-devel" \
target="_blank">https://mail.kde.org/mailman/listinfo/plasma-devel</a><br> \
</div></div></blockquote></div><br></div></div>



_______________________________________________
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