<table><tr><td style="">rrosch created this revision.<br />rrosch added a reviewer: Kate.<br />rrosch added a project: Kate.<br />Herald added a subscriber: kwrite-devel.<br />rrosch requested review of this revision.
</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><strong>REVISION SUMMARY</strong><div><p>I really like the new session manager dialog (session chooser) but when I went to click on the column headers to sort, it didn't work. So I wondered how hard it would be to implement and just went for it.</p>

<p>I know the diff is supposed to be done with arc, but I don't yet have that all set up, and I wanted to get you the patch over as soon as possible. I hope that's ok.</p>

<p>One thing that does not work properly is the sorting by number of open files, since that requires numeric sort which was beyond my current abilities, maybe someone else can take that up for the next round, once this simple patch is in.</p>

<p>I also corrected a couple spelling mistakes, which accounts for the extra lines.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Apply patch (should apply cleanly to most recent master).<br />
Compile.<br />
Run kate.</p></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>AFFECTED FILES</strong><div><div>kate/session/katesessionchooseritem.h<br />
kate/session/katesessionmanagedialog.cpp<br />
kate/session/katesessionmanagedialog.h<br />
kate/session/katesessionmanagedialog.ui</div></div></div><br /><div><strong>To: </strong>rrosch, Kate<br /><strong>Cc: </strong>kwrite-devel, rrosch, domson, michaelh, ngraham, demsking, cullmann, sars, dhaumann<br /></div>