<table><tr><td style="">leinir created this revision.<br />leinir added reviewers: Kirigami, Discover Software Store.<br />leinir added a project: Kirigami.<br />Restricted Application added a subscriber: plasma-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/D9816" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>The simple QML based Icon already provides this correctly, but DesktopIcon would not output in the correct location and aspect ratio, and rather draw the image scaled into the entirety of the node rectangle.</p>

<p>This fixes the issue described in <a href="https://bugs.kde.org/show_bug.cgi?id=388737" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=388737</a></p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Without this patch, Discover shows items as seen in the attachment on the bug</p>

<p>With this patch, the output is as expected (with the correct aspect ratio and centered in the item)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R169 Kirigami</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D9816" rel="noreferrer">https://phabricator.kde.org/D9816</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/desktopicon.cpp</div></div></div><br /><div><strong>To: </strong>leinir, Kirigami, Discover Software Store<br /><strong>Cc: </strong>plasma-devel, apol, davidedmundson, mart, hein<br /></div>