<table><tr><td style="">bruns 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/D20096">View Revision</a></tr></table><br /><div><div><p>I think the code itself is good to go now.</p>

<p>Can you please update the summary somewhat (and better do it directly in phabricator, as arc will throw away any changes in the commit message, unless you do a  - <tt style="background: #ebebeb; font-size: 13px;">arc amend</tt>, update the commit message in git, <tt style="background: #ebebeb; font-size: 13px;">arc diff --verbatim</tt> - cycle).</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">describe the dolphin screenshot textually</li>
<li class="remarkup-list-item">update the FIXED-IN, unless it lands before KF 5.57 is tagged</li>
<li class="remarkup-list-item">in the test plan, say what <strong>is</strong> tested</li>
<li class="remarkup-list-item">probably better first sentence:</li>
</ol>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>The birthtime can be retrieved using using the statx function, available since glibc 2.28. In case the kernel lacks support for the underlying syscall, glibc falls back to stat internally. The validity of the btime field is indicated in the mask field, e.g. when the kernel or the filesystem lacks support for btime. For details, see <manpage url>.</p></blockquote></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20096">https://phabricator.kde.org/D20096</a></div></div><br /><div><strong>To: </strong>meven, Frameworks, dfaure, fvogt, bruns, broulik<br /><strong>Cc: </strong>pino, bcooksley, ngraham, kde-frameworks-devel, michaelh, bruns<br /></div>