<table><tr><td style="">harogaston 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/D29115">View Revision</a></tr></table><br /><div><div><p>Hi all!</p>

<p>Unfortunately I'm not finding time to do this properly. I tried but my machine has changed and seems that I cannot compile and test the projects anymore.<br />
I'll have to look into that eventually but I certainly cannot do it now.</p>

<p>Given the latest feedback this is all there is to change I believe:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="diff" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);"><span style="color: #000080">diff --git a/src/kitemviews/kfileitemmodel.cpp b/src/kitemviews/kfileitemmodel.cpp</span>
<span style="color: #000080">index dbbd63a6a..fe502fbb3 100644</span>
<span style="color: #a00000">--- a/src/kitemviews/kfileitemmodel.cpp</span>
<span style="color: #00a000">+++ b/src/kitemviews/kfileitemmodel.cpp</span>
<span style="color: #800080">@@ -1713,6 +1713,15 @@ bool KFileItemModel::lessThan(const ItemData* a, const ItemData* b, const QColla</span>
         }
     }

<span style="color: #00a000">+    // Show hidden files and folders last</span>
<span style="color: #00a000">+    const bool isHiddenA = a->item.isHidden();</span>
<span style="color: #00a000">+    const bool isHiddenB = b->item.isHidden();</span>
<span style="color: #00a000">+    if (isHiddenA && !isHiddenB) {</span>
<span style="color: #00a000">+        return false;</span>
<span style="color: #00a000">+    } else if (!isHiddenA && isHiddenB) {</span>
<span style="color: #00a000">+        return true;</span>
<span style="color: #00a000">+    }</span>
<span style="color: #00a000">+</span>
     if (m_sortDirsFirst || m_sortRole == SizeRole) {
         const bool isDirA = a->item.isDir();
         const bool isDirB = b->item.isDir();</pre></div>

<p>If anyone can push that change upstream, via Gitlab or any other means please fell free.</p>

<p>Sorry about the inconvenience.</p>

<p>Best,</p>

<p>Gastón</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R318 Dolphin</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D29115">https://phabricator.kde.org/D29115</a></div></div><br /><div><strong>To: </strong>harogaston, Dolphin, ngraham, elvisangelaccio<br /><strong>Cc: </strong>meven, kfm-devel, badbunny, waitquietly, azyx, nikolaik, pberestov, iasensio, aprcela, fprice, fbampaloukas, alexde, Codezela, feverfew, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp, rdieter, mikesomov<br /></div>