<table><tr><td style="">leinir created this revision.<br />leinir added reviewers: KNewStuff, ngraham, sitter.<br />leinir added a project: KNewStuff.<br />Herald added a project: Frameworks.<br />Herald added a subscriber: kde-frameworks-devel.<br />leinir requested review of this revision.
</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/D20693">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>In the before-times, drop shadows were considered vitally important on everything, and so it was introduced to KNewStuff's dialogues as well. It was, however, never done quite right, and it's making our dialogues look all silly. So, we remove it for now, and if we still want it, we can reintroduce them in a more modern fashion.<br />
BUG:391108</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Open the details dialogue of any knewstuff listing<br />
Before patch: See pixelated (and weirdly positioned) pixelated border on all preview images<br />
After patch: See no pixelated border</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R304 KNewStuff</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20693">https://phabricator.kde.org/D20693</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>data/CMakeLists.txt<br />
data/thumb_frame.png<br />
src/ui/imagepreviewwidget.cpp<br />
src/ui/imagepreviewwidget_p.h</div></div></div><br /><div><strong>To: </strong>leinir, KNewStuff, ngraham, sitter<br /><strong>Cc: </strong>kde-frameworks-devel, KNewStuff, michaelh, ZrenBot, ngraham, bruns<br /></div>