Changes to Plasma Weather - Git revision 5a09cd815edd7723800c0ba43a8be9f67d28e011

[Shawn Starr]

On Tuesday, September 11, 2012 01:30:50 PM David Edmundson wrote:
> On Tue, Sep 11, 2012 at 1:05 PM, Shawn Starr <shawn.starr at rogers.com> wrote:
> > On Tuesday, September 11, 2012 08:30:50 AM Luís Gabriel wrote:
> >> 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ís Gabriel
> > 
> > FYI, this discussion has been taken offline to IRC right now.
> 
> Make sure someone posts a summary here afterwards so history is kept.
> 
> > Thanks,
> > Shawn.
> > 

The changes are fine, it's a 1:1 exact replication from the C++ code to 
QML/Javascript.

Since Plasma isn't planning on changing the UX/UI guidelines, I'll continue to 
help with this new QML ported one.

It will mean I will diverge from the QML ported applet and turn this into a 
full-fledged QML application (media center like) with more information and 
animations and a ticker. This will resemble what you might see on Weather 
channel/The Weather Network local forecast updates

I designed the backend API especially for the other rich weather data, like 
sunrise/sunset, and would show animation of sunrise/sunset w/ QML with nice 
'page' transitions for the different weather information.

A lot of coding to do once Luis is done converting applet to QML.

Thanks,
Shawn.

> >> On Tue, Sep 11, 2012 at 8:15 AM, Shawn Starr <shawn.starr at rogers.com> 
wrote:
> >> > On Tuesday, September 11, 2012 07:58:16 AM Luís 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
> >> > > the
> >> > > current master.
> >> > > 
> >> > > Cheers,
> >> > > --
> >> > > Luís 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 at gmail.com> 
wrote:
> >> > > > On Tuesday 11 September 2012, Shawn Starr wrote:
> >> > > > > Hi Luís,
> >> > > > > 
> >> > > > > 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
> >> > > > 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
> >> > 
> >> > 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
> >> > > > 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 at kde.org
> >> > > > https://mail.kde.org/mailman/listinfo/plasma-devel
> > 
> > _______________________________________________
> > Plasma-devel mailing list
> > Plasma-devel at kde.org
> > https://mail.kde.org/mailman/listinfo/plasma-devel
> 
> _______________________________________________
> Plasma-devel mailing list
> Plasma-devel at kde.org
> https://mail.kde.org/mailman/listinfo/plasma-devel