<table><tr><td style="">hein accepted this revision.<br />hein added a comment.<br />This revision is now accepted and ready to land.
</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/D11358">View Revision</a></tr></table><br /><div><div><p>Folder View always reserves enough vertical space to fit the configured number of lines to keep an evenly-spaced grid. The default number of lines should therefore strike a happy medium between conserving vertical space to fit more rows and catering to the odd longer file name. 3 would optimize for most file names being so long that even with ellision they're not readable at two lines, which I don't feel is warranted.</p>
<p>As for making it configurable - maybe. Not sure. I'm not there yet, because on my end, the user complaints have stopped with 4.10 when we prominently followed feedback and made the delegates and grid tighter. Before that we got tickets about it more regularly, which has mostly ceased, and the release itself was met with lots of positive feedback. I therefore feel that we've been doing quite well at iterating towards what makes most people happy. I'm sure it's always possible to find individual opinions that buck the trend, but I have to integrate the feedback channels somehow.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>BRANCH</strong><div><div>more-generous-folder-view-title-display (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D11358">https://phabricator.kde.org/D11358</a></div></div><br /><div><strong>To: </strong>ngraham, hein, Plasma<br /><strong>Cc: </strong>plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>