<table><tr><td style="">ndavis added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D25820">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
color: #6B748C;
font-style: italic;
margin: 4px 0 12px 0;
padding: 8px 12px;
background-color: #F8F9FC;">
<div style="font-style: normal;
padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D25820#574397" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D25820#574397</a>, <a href="https://phabricator.kde.org/p/vinzenzv/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@vinzenzv</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><blockquote style="border-left: 3px solid #8C98B8;
color: #6B748C;
font-style: italic;
margin: 4px 0 12px 0;
padding: 8px 12px;
background-color: #F8F9FC;">
<div style="font-style: normal;
padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D25820#574119" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D25820#574119</a>, <a href="https://phabricator.kde.org/p/ndavis/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@ndavis</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><p>Good start! The lens flare looks upside down</p></div>
</blockquote>
<p>That was intentionally to make it distinguishable from digiKam.</p></div>
</blockquote>
<p>I don't think it works because the light source in Breeze comes from the top left.</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>and the icon needs more pixel alignment.</p></blockquote>
<p>I didn't move anything around besides the lense flares. So consequentially the digiKam would need that as well. I'm not sure if that's intended?</p></blockquote>
<p>It's the lens flare and center part that aren't pixel aligned. Except for the smaller dot in the lens flare, they were aligned in the original version. <a href="https://phabricator.kde.org/F7807704" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F7807704: Screenshot_20191210_075015.png</a><br />
What you could do instead of a lens flare is implement the rings of light that the official icon uses: <a href="https://github.com/MaartenBaert/ssr/blob/master/data/icons/48/simplescreenrecorder.png" class="remarkup-link" target="_blank" rel="noreferrer">https://github.com/MaartenBaert/ssr/blob/master/data/icons/48/simplescreenrecorder.png</a><br />
It doesn't have to be exactly the same as the original's rings, but it should remind you of them and look fairly natural.</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>I feel like the red/green/blue colors of the center area are a bit too dark as well.</p></blockquote>
<p>Yes, I'd thinks so too - but I took the original icon's colors. So we'd better not change that or the icon would move to far from the original?</p></blockquote>
<p>It's OK to deviate a bit from the original as long as you preserve the spirit of the original.</p>
<p>In case you haven't seen the HIG, here is a list of some official colors you can use: <a href="https://hig.kde.org/style/color/default.html" class="remarkup-link" target="_blank" rel="noreferrer">https://hig.kde.org/style/color/default.html</a><br />
You can deviate from that list, but be mindful of the hues and saturation levels so that your choice of color isn't too far from what Breeze normally looks like. Also, avoid using any of the "Primary" colors (excluding Icon Gray, which we don't actually use for icons much) for backgrounds unless you have an outline in a different color. You don't want your icon to blend into the background of the UI.</p>
<hr class="remarkup-hr" />
<p>An extra tip: Zoom out to 100% with the grid, guides and page outline turned off so you can see what your icon would normally look like. It's important to make sure it looks good at higher zoom levels too since SVGs are scalable, but you should be most concerned with how it looks at 100% size.</p>
<p>As you can see, even though there is an inner ring and a smaller circle, it looks like one dark green circle because they aren't pixel aligned. The smaller circle in the lens flare looks a bit shorter than it actually is for the same reason.<br />
<a href="https://phabricator.kde.org/F7808128" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F7808128: Screenshot_20191210_114611.png</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R266 Breeze Icons</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D25820">https://phabricator.kde.org/D25820</a></div></div><br /><div><strong>To: </strong>vinzenzv, VDG, ndavis<br /><strong>Cc: </strong>filipf, ndavis, kde-frameworks-devel, LeGast00n, fbampaloukas, GB_2, trickyricky26, michaelh, ngraham, bruns<br /></div>