Current track applet changes. Was: Re: extragear/multimedia/amarok/src/context/applets/currenttrack

[Nikolaj Hald Nielsen]

> This isn't just at you, but I'm replying here because its the latest occurance.
>
> Can we please stop the "make pretty" commits?  Artwork is so
> subjective that I'm fairly sure this is at least the fourth time we've
> had a "make pretty" commit for the current track applet.
> Before making visual changes (especially in what is essientally a
> freeze) can we please have discussion on the mailing list?  This is
> starting to get riddiculous.

I sort of agree with the gist of your comment, but then not really
anyway. This commit fixed some very obvious layout and visibility
issues. For instance you can now actually read the text for score,
play count and last played ( its not on top of the icons any more ).
Sure, there was some aesthetic changing of layouts and new icons that
can be very subjective, but whenever we try to discuss these things it
ends up with nobody agreeing on anything ( these things are hard to
visualize ) so I commit them so people can see them an make up their
opinion from there. Mainly, as Ian points out, as that is how we have
always done things.

Now, for the discussion of subjective stuff. I like the idea of the
new "non black" icons, especially the new "play count" one ( although
some of these icons are rendered horribly by QtSvg, I will need to
have a look at this and possible export them to PNG and include them
in the svg ). Some of the icons needs work still though. I also sort
of like the layout now. I was also considering having the 3 additional
info labels and their icons along the bottom next to the rating
widget, but I think the current way makes more sense. If I can fix the
rendering issues, I am quite happy with how it looks now, at least for
2.0.0

- Nikolaj