broulik updated this revision to Diff 9658. broulik added a comment. Restricted Application added a subscriber: Frameworks.
- Don't blindly round both sizes, instead round one and then scale the other accordingly - Adjust unit test REPOSITORY R242 Plasma Frameworks CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3892?vs=9577&id=9658 REVISION DETAIL https://phabricator.kde.org/D3892 AFFECTED FILES autotests/iconitemtest.cpp autotests/iconitemtest.h src/declarativeimports/core/iconitem.cpp src/declarativeimports/core/iconitem.h EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: broulik, #plasma, hein Cc: #frameworks, mart, davidedmundson, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas