[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 10:58:16
Message-ID: CANV84AP=FRArinV4RSJ8Sz=NimFK8BNuhN1W5zh06YkB5KHJRA () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


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 the
current master.

Cheers,
--
Lu=EDs Gabriel


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 nic=
e
> 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 t=
he
> > 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 unfortunate
> 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 wil=
l
> soon share the same fate)
>
> about the 2.0 you have in mind, the current design well fits in the globa=
l
> style (simple display of information that cuts useless graphic elements t=
o
> the
> minimum) that is the style the default plasmoids should have.
>
> However, if you want something different, you are more than welcome and
> encouraged to work on another plasmoid (and then using the same review
> 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)]

Hi 
<span style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)">Shawn,</span><div><font \
color="#222222" face="arial, sans-serif"><br></font></div><div><font color="#222222" \
face="arial, sans-serif">I don&#39;t see the point of revert the \
changes.</font></div>

<div><font color="#222222" face="arial, sans-serif">Sorry about the misunderstanding \
but as you see in the previous replies, not everyone were aware of your plans to work \
on this plasmoid.</font></div><div><font color="#222222" face="arial, \
sans-serif">Feel free to work on a new UI if you want, but please do it on top of the \
current master.</font></div>

<div><font color="#222222" face="arial, sans-serif"><br></font></div><div><font \
color="#222222" face="arial, sans-serif">Cheers,<br clear="all"></font>--<br>Luís \
Gabriel<br> <br><br><div class="gmail_quote">On Tue, Sep 11, 2012 at 5:55 AM, Marco \
Martin <span dir="ltr">&lt;<a href="mailto:notmart@gmail.com" \
target="_blank">notmart@gmail.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="HOEnZb"><div class="h5">On Tuesday 11 September 2012, Shawn Starr \
wrote:<br> &gt; Hi Luís,<br>
&gt;<br>
&gt; I appreciate you making changes to the code, but it would have been nice to<br>
&gt; been told about this since I am about to replace this with QML porting.<br>
&gt;<br>
&gt; Please discuss with me what changes you would like to make as I am in the<br>
&gt; process of porting to QML for the applet. My goals for &#39;2.0&#39; are \
different<br> &gt; from what you&#39;re doing.<br>
&gt;<br>
&gt; I would like to revert this out of trunk or discuss this on IRC please.<br>
<br>
</div></div>ok, let me be clear about this.<br>
there won&#39;t be any revert.<br>
<br>
all the changes were discussed and toroughly reviewed. It&#39;s unfortunate that<br>
you didn&#39;t participate in it and yes, it&#39;s also unfortunate that we \
didn&#39;t<br> contact you.<br>
<br>
when the rewrite started, that applet was effectively abandoned and bitrotting<br>
since at least two years, as unfortunately are many other applets in<br>
kdeplasma-addons, and this requires an action (other addons plasmoids will<br>
soon share the same fate)<br>
<br>
about the 2.0 you have in mind, the current design well fits in the global<br>
style (simple display of information that cuts useless graphic elements to the<br>
minimum) that is the style the default plasmoids should have.<br>
<br>
However, if you want something different, you are more than welcome and<br>
encouraged to work on another plasmoid (and then using the same review process<br>
that was used in this case)<br>
<br>
Cheers,<br>
Marco Martin<br>
<div class="HOEnZb"><div \
class="h5">_______________________________________________<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> \
</div></div></blockquote></div><br></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