<table><tr><td style="">ngraham 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/D12077">View Revision</a></tr></table><br /><div><div><p>Yes, when the filename or metadata are the primary tools for differentiating files, having the items presented in a list format is much much better, as in your example of browsing for source files. But keep in mind that source files and other textual documents are only one use case! Large icons with previews are often much more helpful for predominately visual content, like images (think of Gwenview's open and save dialogs!), videos, presentations, epubs and comic book files, and many PDFs. This distinction is why I wanted to make it easier to change view modes within the file dialogs, and why I think each of the two view modes accessible from the toolbar should ideally support one of those use cases. If we don't change the icon positioning default, then we'll have two view modes that are optimized for textual content, and none for visual content.</p>

<p>Thankfully, previews aren't shown for tiny icons, and each view mode remembers its own size settings, so it's feasible with very little work to have <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Detailed Tree View</span></span></span> show 16px icons and a nice tight list, and icons-on-top <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Short View</span></span></span> show 48px (or larger) icons with previews. With this, we would have two views, each optimized for one of the two primary content types, which seems humane and appropriate given the diversity of software out there.</p>

<p>Of course this discussion is relevant to the <em>next</em> patch, not this one...</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>BRANCH</strong><div><div>arcpatch-D12077</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D12077">https://phabricator.kde.org/D12077</a></div></div><br /><div><strong>To: </strong>ngraham, Frameworks, rkflx, VDG, abetts<br /><strong>Cc: </strong>rkflx, mmustac, broulik, michaelh, ngraham, bruns<br /></div>