davidedmundson created this revision. davidedmundson added a reviewer: Plasma. Restricted Application added projects: Plasma, Frameworks. Restricted Application added subscribers: Frameworks, plasma-devel.
REVISION SUMMARY Icon's are generally small, they may as well go in the existing shared texture atlas, save some memory and it means more items can be batched whilst rendering. The textures used during animation are unchanged for now as it also requires changing FadingNode's shader. TEST PLAN Ran plasma. We use atlas's in frame/SVGItem already so it should be fine. REPOSITORY R242 Plasma Framework (Library) BRANCH master REVISION DETAIL https://phabricator.kde.org/D4342 AFFECTED FILES src/declarativeimports/core/iconitem.cpp EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: davidedmundson, #plasma Cc: plasma-devel, #frameworks, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
