pereira.alex updated this revision to Diff 78755. pereira.alex added a comment.
Use single slider and sync values Removed duplicated slider, to only use one slider that uses and updates the correct value. Added an event when changing viewmode grid or list, to load the slider with the correct value. If there isn't any issue with the code and behaviour, I think its ready: - adds feature to allow choosing size on list mode - fixes the bug reported in that the list mode icon size updates properly without needing a plasmashell restart REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D28057?vs=78238&id=78755 BRANCH fix-folderview-popup-icon-list-size (branched from master) REVISION DETAIL https://phabricator.kde.org/D28057 AFFECTED FILES containments/desktop/package/contents/config/main.xml containments/desktop/package/contents/ui/ConfigIcons.qml containments/desktop/package/contents/ui/FolderView.qml To: pereira.alex, #plasma, #vdg, ngraham Cc: ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
