<div dir="ltr">Thanks for the feedback. Given the collection of responses so far, I'll go ahead and leave the white outlines as they are.<div><br></div><div>Regarding duplication of svgs. Yes, the changes I prepared had separate panel-background, dialog and tooltip svgs specifically for the dark theme. The main reason is that reflective properties of the edge of the material changes with color - brighter colors reflect more light, darker colors reflect less. So where as the Cardboard Grey of the normal Breeze theme needed a much brighter outline to better represent the quality of the edge, the dark grey of the breeze dark theme would need a much less bright outline. The current white outlines that we've grown to like are a happy accident of the much brighter outline needed for the normal theme. Now, we'll just say that those outlines are intentional. ;-)</div><div><br></div><div>In any event, I think there may be only so much complexity that's worth it to avoid duplication of svgs. Beyond a certain point a theme of a different color is, in the end, just a different theme. And these themes are a couple of drunken grizzly bears to put together as is. :-)</div><div><br></div><div>Thanks much for the feedback! </div><div>Andrew</div><div>[back to visual bug hunting]<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Oct 2, 2014 at 2:25 AM, Marco Martin <span dir="ltr"><<a href="mailto:notmart@gmail.com" target="_blank">notmart@gmail.com</a>></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 Wednesday 01 October 2014 16:21:03 Andrew Lake wrote:<br>
> For anyone using the breeze dark theme, I've been meaning for a while to<br>
> make some adjustments so that the edges aren't quite so pronounced and high<br>
> contrast. It looks this way because it's currently using the same svgs (and<br>
> edge highlights) from the normal breeze theme svgs.<br>
><br>
> I have changes ready to push to reduce that high contrast edge outline, but<br>
> I wanted to check to see if there was anyone that was particularly wedded<br>
> to the those white outlines before I do. I'd do up a screen shot but just<br>
> imagine those bright white outlines turned all the way down.<br>
><br>
> Holler if anyone has any objections. :-)<br>
<br>
</div></div>does the change involve duplicating the svgs?<br>
<br>
I quite like them as well, also i prefer duplication of images to be as little<br>
as possible (just yesterday i fixed some graphics glitches exactly in that,<br>
means next time work gets to be done double of the times)<br>
<br>
I am not that opposed to it, but if it has to be done i would prefer a way<br>
that doesn't duplicate svgs (that should be possible)<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Marco Martin<br>
</font></span><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>