<table><tr><td style="">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><blockquote style="border-left: 3px solid #8C98B8;
color: #6B748C;
font-style: italic;
margin: 4px 0 12px 0;
padding: 8px 12px;
background-color: #F8F9FC;">
<div style="font-style: normal;
padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D28057#628853" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D28057#628853</a>, <a href="https://phabricator.kde.org/p/ngraham/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@ngraham</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><p>Thanks!</p>
<p>Now we have a new problem though: the default icon size for list view is synchronized with the default icon size for icon view, so it's now gigantic by default: <a href="https://phabricator.kde.org/F8180048" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F8180048: Screenshot_20200316_190912.png</a></p>
<p>We need for the default value itself to be different for icon vs list view, while allowing each one to be independent. Probably the cleanest way to do this is to define a new value in the config file for the default list view icon size (could be called something like <tt style="background: #ebebeb; font-size: 13px;">listViewIconSize</tt>) and then change <tt style="background: #ebebeb; font-size: 13px;">makeIconSize()</tt> (good thing you kept it, lol) to return <tt style="background: #ebebeb; font-size: 13px;">return FolderTools.iconSizeFromTheme(plasmoid.configuration.listViewIconSize)</tt> when it's in list view.</p>
<p>Does that make sense?</p></div>
</blockquote>
<p>Yes, it does !<br />
I will look into it, and do like you said, a new listViewIconSize default !</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</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>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>