<table><tr><td style="">dhaumann 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/D27180">View Revision</a></tr></table><br /><div><div><p>I'd be fine with committing this. Still, there very clearly is large room to improve this further.</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D27180#inline-153764">View Inline</a><span style="color: #4b4d51; font-weight: bold;">katesessionchooseritem.h:38</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; ">        <span class="n">setText</span><span class="p">(</span><span style="color: #601200">1</span><span class="p">,</span> <span class="n">docs</span><span class="p">);</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">        <span class="n">setText</span><span class="p">(</span><span style="color: #601200">2</span><span class="p">,</span> <span class="n">s</span><span style="color: #aa2211">-></span><span class="n">timestamp</span><span class="p">().</span><span class="n">toString</span><span class="p">(</span><span class="n">QString</span><span style="color: #aa2211">::</span><span class="n">fromStdString</span><span class="p">(</span><span style="color: #766510">"yyyy-MM-dd  hh:mm:ss"</span><span class="p">)));</span>
</div><div style="padding: 0 8px; margin: 0 4px; ">    <span class="p">}</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">I believe that seconds are a bit over engineered. In fact, it would be nice to do the sorting based on the timestamp as now, but visually the English format does not fit very well in other languages. So this should be translated via i18n. But then we loose the correct sorting.</p>

<p style="padding: 0; margin: 8px;">So for now this patch is ok, but in the long run we should do better.</p>

<p style="padding: 0; margin: 8px;">For instance, I think it would be much more user friendly to have "Yesterday", "3 days ago", ... instead of the datetime right now.</p>

<p style="padding: 0; margin: 8px;">Would you be interested to look into this as well?</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R40 Kate</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D27180">https://phabricator.kde.org/D27180</a></div></div><br /><div><strong>To: </strong>rrosch, Kate, dhaumann, cullmann<br /><strong>Cc: </strong>kwrite-devel, rrosch, domson, michaelh, ngraham, demsking, cullmann, sars, dhaumann<br /></div>