D28057: Fix/Allow folderview popup mode icon and list icon size

Alexandre Pereira noreply at phabricator.kde.org
Sat Mar 28 20:10:12 GMT 2020


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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200328/f0ec720e/attachment.html>


More information about the Plasma-devel mailing list