<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#645164" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D28057#645164</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>I would give each view mode its own config value and adjust the code with conditionals; that's not a problem. There are no backwards compatibility concerns here if you do that, except for people currently using list view with gigantic icons, which will become small, fixing the bug. If they complain, we can just tell them that it was a bug and if they want huge list icons again, they'll have to configure them that way deliberately. :)</p>

<p>Alternatively we can give up on allowing the icon size in list view to be configured and just always use a small size.</p></div>
</blockquote>

<p>Sorry, maybe i didn't explain it very well: that was like I had previously, but in that way, it would actually not fix the bug. the bug is that the rest of the plasmoid relies on a default size value. ( which I believe was the original bug report and what I wanted to fix ).</p>

<p>Maybe its just coding styles/schools, but I honestly think its better this way, update the size when configuring and then use default size everywhere, than adding bunch of conditions through out the code, being error prone.</p>

<p>but well, if it must be so... then ok</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>