<table><tr><td style="">pgkos updated this revision to Diff 19592.<br />pgkos changed the repository for this revision from R120 Plasma Workspace to R242 Plasma Framework (Library).<br />pgkos added a comment.<br />Restricted Application added a project: Frameworks.<br />Restricted Application added a subscriber: Frameworks.
</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/D7849" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Edited the function <tt style="background: #ebebeb; font-size: 13px;">Units::roundToIconSize</tt> so it both accepts and outputs a dpi-scaled pixel value.</p>

<p>This might break some things, but on the other hand there are places (other than the system tray) which also have exactly the same problem with icon scaling on hidpi screens (see e.g. <a href="https://cgit.kde.org/plasma-desktop.git/tree/applets/taskmanager/package/contents/ui/Task.qml" class="remarkup-link" target="_blank" rel="noreferrer">https://cgit.kde.org/plasma-desktop.git/tree/applets/taskmanager/package/contents/ui/Task.qml</a>), this patch will also fix all these other cases.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R242 Plasma Framework (Library)</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D7849?vs=19585&id=19592" rel="noreferrer">https://phabricator.kde.org/D7849?vs=19585&id=19592</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7849" rel="noreferrer">https://phabricator.kde.org/D7849</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/declarativeimports/core/units.cpp</div></div></div><br /><div><strong>To: </strong>pgkos, Plasma<br /><strong>Cc: </strong>Frameworks, davidedmundson, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>