<table><tr><td style="">pereira.alex updated this revision to Diff 78758.<br />pereira.alex added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D28057">View Revision</a></tr></table><br /><div><div><p>Deal with upgrading configuration</p>

<p>Added code to deal with old configuration:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">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 )</li>
<li class="remarkup-list-item">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.</li>
</ul>

<p>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.<br />
You think it is worth it ? or don't check it and let users handle migration ?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D28057?vs=78755&id=78758">https://phabricator.kde.org/D28057?vs=78755&id=78758</a></div></div><br /><div><strong>BRANCH</strong><div><div>fix-folderview-popup-icon-list-size (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D28057">https://phabricator.kde.org/D28057</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>containments/desktop/package/contents/config/main.xml<br />
containments/desktop/package/contents/ui/ConfigIcons.qml<br />
containments/desktop/package/contents/ui/FolderView.qml</div></div></div><br /><div><strong>To: </strong>pereira.alex, Plasma, VDG, ngraham<br /><strong>Cc: </strong>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<br /></div>