<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/126319/">https://git.reviewboard.kde.org/r/126319/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On December 11th, 2015, 7:08 p.m. UTC, <b>Martin Klapetek</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;"><blockquote style="text-rendering: inherit;padding: 0 0 0 1em;border-left: 1px solid #bbb;white-space: normal;margin: 0 0 0 0.5em;line-height: inherit;">
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Attached images directly to request; if this is not the preferred way of attaching large files</p>
</blockquote>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Attaching just one file would be fine, I mean this is just to see it right?
So there's no need to upload /all/ the resolutions, imo :)</p>
<blockquote style="text-rendering: inherit;padding: 0 0 0 1em;border-left: 1px solid #bbb;white-space: normal;margin: 0 0 0 0.5em;line-height: inherit;">
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I would like to shift to more "professional" default wallpapers</p>
</blockquote>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Can you clarify a bit what does that mean?</p>
<hr style="text-rendering: inherit;margin: 0;padding: 0;white-space: normal;border: 1px solid #ddd;line-height: inherit;" />
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Overall the wallpaper strikes me as a visual corruption. This may be due
to my currently slightly broken intel driver which creates all sorts of
graphical glitches and that wallpaper to me just looks like it glitched
(the shadow polygon going on from the top-left corner is just a strong
reminescence of that plus the "rainbow noise" going on at the lower-right
diagonal part of the image).</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;">Ah, sorry, haven't seriously used reviewboard. Wasn't sure if the content in the request would be what actually goes into the repo. David very patiently explained everything I needed to know about this all, but my birdy little brain promptly forgot anything important and I don't think I've (properly) gotten as far as having a patch committed. Sorry - this is an area I'm trying to improve.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Generally when I talk about "professional" I'm talking less organic, wild, "in your face" wallpapers. 5.2 was just horrible neon mess, 5.3 looked cluttered, 5.4 was regularly called an "acid trip", etc... I don't want to make things dull and lifeless, but after going down the current road I think a cleaner approach is worth a shot. Practically this means I want to be more conservative in the shapes and clutter department, rope in the organics a bit, and focus on clearly defined visually interesting layers.</p>
<hr style="text-rendering: inherit;margin: 0;padding: 0;white-space: normal;border: 1px solid #ddd;line-height: inherit;" />
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Hopefully our users aren't exposed to enough visual corruption to feel that way :P </p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">But in all seriousness I will move the top-left of the "shadow polygon" away from the corner though (possibly hiding it off-screen entirely (in certian sizes it is cropped anyway), I do regularly work on a machine with AMD graphics w/ graphical bugs and I can understand the where you're coming from when I specifically look for it. </p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Otherwise people who have seen this wallpaper haven't mentioned it looking glitchy, so general users may not be as sensitive to this. Consider that I personally don't see it as a big issue, as for some "glitchy" is even a desirable look. Before I remove/alter the circuit pattern I'd like to see another person or two weigh in and answer if I'm biased because I know what it's supposed to be, or if you are possibly biased from working on a system with visual corruption. Does anyone else think;
- the pattern could be regularly/easily misidentified as visual corruption?
- ... And would a wallpaper with that style be a problem?</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I'll wait until we decide what to do with the pattern before loading another version (with the shadow polygon altered)</p></pre>
<br />
<p>- Ken</p>
<br />
<p>On December 11th, 2015, 7:04 p.m. UTC, Ken Vermette 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 Ken Vermette.</div>
<p style="color: grey;"><i>Updated Dec. 11, 2015, 7:04 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
breeze
</div>
<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;">Attached images directly to request; if this is not the preferred way of attaching large files, please let me know and I'll change methods, perhaps using the VDG share if that's preferred.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">For reference I would like to shift to more "professional" default wallpapers and away from the more organic style of 5.4 and 5.5. Feedback on both general direction or this wallpaper is welcome.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Note: I will submit splash files later in case of changes to this wallpaper.</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;">Dogfooded on several machines and my phone for a couple weeks now, doesn't burn eyes out. Corner isn't as distracting as one might assume, especially once a window is on screen. Got early feedback, adjustments made from origional in order:
- Re-drew the "circuit/chip" pattern to be less noisy/distracting/cleaner. Adjusted it so details are still seen on lower-res wallpapers.
- Lightened purple to avoid the "dark void" feeling people have mentioned feeling from 5.5 wallpaper.
- Fixed minor pixel issues for triangles near edge
- Adjusted noise, shadows.</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;">
<li>wallpapers/Next/contents/images/1024x768.png <span style="color: grey">(60e1205)</span></li>
<li>wallpapers/Next/contents/images/1280x1024.png <span style="color: grey">(36a9130)</span></li>
<li>wallpapers/Next/contents/images/1280x800.png <span style="color: grey">(c33e594)</span></li>
<li>wallpapers/Next/contents/images/1440x900.png <span style="color: grey">(2c75b54)</span></li>
<li>wallpapers/Next/contents/images/1600x1200.png <span style="color: grey">(5ddaf72)</span></li>
<li>wallpapers/Next/contents/images/1638x1024.png <span style="color: grey">(a3c7492)</span></li>
<li>wallpapers/Next/contents/images/1680x1050.png <span style="color: grey">(eddc47e)</span></li>
<li>wallpapers/Next/contents/images/1920x1080.png <span style="color: grey">(ab6d950)</span></li>
<li>wallpapers/Next/contents/images/2560x1440.png <span style="color: grey">(5c78e9d)</span></li>
<li>wallpapers/Next/contents/images/2560x1600.png <span style="color: grey">(eeb08a1)</span></li>
<li>wallpapers/Next/contents/images/3200x1800.png <span style="color: grey">(7340567)</span></li>
<li>wallpapers/Next/contents/images/3200x2000.png <span style="color: grey">(fd1a62c)</span></li>
<li>wallpapers/Next/contents/screenshot.png <span style="color: grey">(a6d2b7b)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/126319/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">File Attachments </h1>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2015/12/11/5e9e5db3-c8d4-43ec-8236-da5daa80be85__1024x768.png">1024x768.png</a></li>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2015/12/11/41e11bb9-eaef-4eb1-bb4b-c7137becaca0__1280x800.png">1280x800.png</a></li>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2015/12/11/ee643bd5-8162-4a9b-ae09-df090d4f0b43__screenshot.png">screenshot.png</a></li>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2015/12/11/054536c9-8160-418a-a1c9-c319e1e2b533__1280x1024.png">1280x1024.png</a></li>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2015/12/11/c4c61ba4-10ea-4d87-9437-7b7e1701eaf7__1440x900.png">1440x900.png</a></li>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2015/12/11/ee746ed5-428d-4423-8e20-d9786b25190f__1600x1200.png">1600x1200.png</a></li>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2015/12/11/233a9950-24bd-47b6-8c7a-8f3e6fe707df__1638x1024.png">1638x1024.png</a></li>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2015/12/11/4c86e45c-eb10-47fa-b1ff-736bd65918cc__1680x1050.png">1680x1050.png</a></li>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2015/12/11/93cc544a-a8a8-4291-91ff-58140ec5ed64__1920x1080.png">1920x1080.png</a></li>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2015/12/11/73ad5caf-5156-495b-939c-387ce4970140__2560x1440.png">2560x1440.png</a></li>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2015/12/11/1ff471aa-2e26-41d9-8921-2c62544d3415__2560x1600.png">2560x1600.png</a></li>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2015/12/11/7590da34-e5c5-4f55-9ac9-0a22c13a9c0d__3200x1800.png">3200x1800.png</a></li>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2015/12/11/229a429e-9633-4988-ad78-dfb039583fc4__3200x2000.png">3200x2000.png</a></li>
</ul>
</td>
</tr>
</table>
</div>
</body>
</html>