<table><tr><td style="">broulik created this revision.<br />broulik added reviewers: dfaure, cfeck, ngraham.<br />Herald added a project: Frameworks.<br />Herald added a subscriber: kde-frameworks-devel.<br />broulik 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/D16291">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>When requesting an icon size that cannot be served, the pixmap is centered in the requested size.<br />
By using the original pixmap's device pixel ratio we ensure the padded pixmap is painted correctly.</p>

<p>BUG: 396990</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>There's still various issues in Dolphin's view engine when using fractional scaling, such as icons being too small, or preview icons larger then file icons, etc but at least icons aren't cut-off and misaligned anymore.<br />
I don't fully understand why for SVGs it fails to get a properly sized pixmap and instead pads them but that's fractional scaling for you right there...</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">deleted icon cache in <tt style="background: #ebebeb; font-size: 13px;">~/.cache</tt>, ran <tt style="background: #ebebeb; font-size: 13px;">QT_SCREEN_SCALE_FACTORS=1.4 dolphin</tt>, no longer have icons rendered out of bounds</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R302 KIconThemes</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D16291">https://phabricator.kde.org/D16291</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/kiconengine.cpp</div></div></div><br /><div><strong>To: </strong>broulik, dfaure, cfeck, ngraham<br /><strong>Cc: </strong>kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>