<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/125568/">https://git.reviewboard.kde.org/r/125568/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On October 9th, 2015, 10:51 a.m. UTC, <b>Marco Martin</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I tried it but unfortunately I have several issues with it:
<em style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;"> the first one, that may have avoided all the others: I discovered the existence of this thing just now. the default plasma theme is the one thing that
I'm sorry this looks like a complete refusal of it :( but again all of those could have been avoided with a different workflow
</em> It doesn't use the stylesheets, colors can't be dynamically changed (of all the issues, I consider this one the absolute showstopper).
<em style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;"> there are embedded images that are used just as references (like in button) this makes the files heavier to load
</em> actionbuttons won't work that way, it will break all round buttons that aren't radio buttons (as the checkmark won't be rendered correctly as well)
<em style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;"> I really don't like shadows of one single pixel, especially in the panel/popups (while i don't like the plasma theme trying too hard to look like the desktop theme, the depth of the shadow was the one thing that should have been either consistent or even a bit more, as they are at an higher z-order than the other woindows)
</em> besides being one single pixel, the shadow is wrong at the corner, the background is one pixel rounded, while the inner part of the shadow is square, leaving an hole of one pixel at the corner. On high dpi displays that would become very visible
* I really, really dislike the taskbar, looks cluttered and somewhat "old"</p></pre>
</blockquote>
<p>On October 9th, 2015, 11:20 a.m. UTC, <b>Marco Martin</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">anyways, I think i'll try to write a simple script or app to batch convert monochrome icons to using stylesheets so even when icons from theme and from plasma are mixed the colors would be correct</p></pre>
</blockquote>
<p>On October 9th, 2015, 3:39 p.m. UTC, <b>Marco Martin</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">made a simple tool that applies the stylesheets to svgs replacing some colors that come from the breeze palette, may still need some adjustments but seems to work.
it needs the "xmlstarlet" commandline tool that should be packaged in most distributions.</p></pre>
</blockquote>
<p>On October 9th, 2015, 8:08 p.m. UTC, <b>Uri Herrera</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I was never fond of having a Plasma theme that looked like it's having it's own party and that the application windows were just invited to that party. Different colors, different buttons, different icons, different direction. It doesn't use stylesheets because here: base.kde.org/Development/Tutorials/Plasma/ThemeDetails that's not mentioned, not once, not a reference and so I don't know how do they work, what each value applies to and playing the guessing game it's not fun. So I went with what I know and what I could figure out on my own. The embedded bitmaps are just leftovers that I haven't removed. I used the plasmoids that are installed by default to test the theme and I haven't found one that uses actionbuttons, the checkmark in itself was wrong it was a tick when it's a square in the widget style it doesn't make sense to me that it's different and it's being rendered inside a button too. I didn't see either a plasmoid that uses the radio buttons. Using heavy shadows is fine for application windows, it's visually fine, but widgets are not applications and it's not appealing to have them use a heavier shadow than the applications.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The taskmanager was absolutely confusing to me the way it is and it was the second thing I changed after the icons.</p></pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">i guess the plasmoids with not much shadow can make sense, i would really like more shadow on panel and popups tough since they go over the windows (and a quite pedestrian practical reason, i want to keep it quite visible as is done in a complicate way passing it to kwin and we are porting the mechanism to wayland, if it becomes almost invisible it may well break without us noticing in the ongoing heavy refactoring)</p></pre>
<br />
<p>- Marco</p>
<br />
<p>On October 9th, 2015, 8:28 a.m. UTC, Uri Herrera wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for Plasma.</div>
<div>By Uri Herrera.</div>
<p style="color: grey;"><i>Updated Oct. 9, 2015, 8:28 a.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">SVG elements done from scratch, reused only a small portion of the existing files. Colors now match Breeze colorscheme. Buttons now match widget style. Icons in the system tray are taken from the icon theme. Icons in Kickoff are not monochromatic as only 16px and 22px icons are monochrome in Breeze. Better padding overall for elements. New visual for both task manager widgets that matches the states of the buttons. Checkboxes, raio buttons, sliders and scrollbars now have the colors used in the Breeze colorscheme.</p></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;"><img alt="Before" src="http://i.imgur.com/Cz2JNee.png" style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
<img alt="After" src="http://i.imgur.com/hiZlyrb.png" style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" /></p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;"><img alt="Elements after" src="http://i.imgur.com/uMFsMzp.png" style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
<img alt="Elements before" src="http://i.imgur.com/pIs1CZ2.png" style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" /></p></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
</ul>
<p><a href="https://git.reviewboard.kde.org/r/125568/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>