pereira.alex updated this revision to Diff 78758.
pereira.alex added a comment.

  Deal with upgrading configuration
  
  Added code to deal with old configuration:
  
  - created a check to see if the user is using list mode view and doesn't have 
listViewIconSize set. If it doesnt, then use small icon size ( which was old 
behaviour )
  - created code to set the slider to correct size when its used the first time 
for users upgrading. It does it by setting the slider value to the current 
value. When saving, it will then deal with saving into the correct value.
  
  I haven't found any problems/bugs so far on my testing ( manually editing the 
applets rc plasma file to fake like if it was an old configuration ). The only 
issue I have with this, is that that check will need to run whenever the 
folderview is opened.
  You think it is worth it ? or don't check it and let users handle migration ?

REPOSITORY
  R119 Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D28057?vs=78755&id=78758

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

Reply via email to