<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 17 July 2017 at 17:08, Dirk Hohndel <span dir="ltr"><<a href="mailto:dirk@hohndel.org" target="_blank">dirk@hohndel.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Jocke,<div><br></div><div>Thanks for all the work on the QML theming. </div><div><br></div><div>One thing that I noticed is that Davide's design used three different colors: primary, light, </div><div>darker - but I don't think we really do that, we only seem to use two. Not sure when/where</div><div>that got lost.</div><div><br></div><div>The other thing that I notice (and that's something that you added) is the coloring of the</div><div>secondary buttons - I don't like that as much. For one thing, the icons are near invisible</div><div>(but that could be fixed by changing the color they are drawn in), but also I thought that</div><div>having just the center button colored actually created a nice contrast.</div></div></blockquote><div><br></div><div>The reason it looks a bit strange at the moment is the Kirigami issue I mentioned in the </div><div>PR description. Currently Kirigami applies the same color overlay to all icons.</div><div>We will need to be able to define the overlay color of each set of icons individually.</div><div>If you switch between blue, pink and dark you will see what I mean.</div><div>The affected buttons are in  the global drawer, and the secondary buttons.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div><br></div><div><img height="320" width="180" id="m_1210685826162498838m_-846971395821393325286424DF4-0874-45FE-B75D-2F67B941197D" src="cid:55337FEA-2263-4671-B506-E20F500AD030"><img id="m_1210685826162498838m_-8469713958213933252208B0D0B-B934-4C6F-9A3F-365FB573E963" height="320" width="180" src="cid:26CD6462-9C88-4C22-B4DD-4815181C3E2D"></div><div><br></div><div>Maybe we could use the lighter color for the side buttons?</div></div></blockquote><div><br></div><div>Yes, the plan is to use the same color for the secondary buttons as the background in the global drawer.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><span class="m_1210685826162498838HOEnZb"><font color="#888888"><div><br></div><div>/D</div></font></span></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="m_1210685826162498838gmail_signature" data-smartmail="gmail_signature"><div>Jocke</div></div>
</div></div>