<table><tr><td style="">broulik updated this revision to Diff 43847.<br />broulik edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-h2lk7luwplubjq7/">(Show Details)</a><br />broulik 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/D16291">View Revision</a></tr></table><br /><div><div><ul class="remarkup-list">
<li class="remarkup-list-item">Also properly create new pixmap</li>
</ul></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;">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.<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"><br />
<br />
There is also a bug in caching and restoring from cache the icon. Initally the icon is fine, then it's cached to disk but when it is restored the icon doesn't match the requested size anymore and is instead padded.</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">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...</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Probably us writing "icon@2x" to cache, then restoring it as "icon@1.4" and then there's a mismatch.. </span><br />
<br />
* deleted icon cache in `~/.cache`, ran `QT_SCREEN_SCALE_FACTORS=1.4 dolphin`, no longer have icons rendered out of bounds</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R302 KIconThemes</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D16291?vs=43845&id=43847">https://phabricator.kde.org/D16291?vs=43845&id=43847</a></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>