<table><tr><td style="">loh.tar updated this revision to Diff 46175.<br />loh.tar 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/D16926">View Revision</a></tr></table><br /><div><div><p>Glad to read that :-)</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">I have grouped the buttons by functionality, and always dislike designs where dangerous functionality is near placed by often used one. So my biggest objection here are the similar looking Delete and Close buttons. That's why are separated by a spacer. But I understand that there are also "Design-Rules" exist to place always the same function in the same order at the same place. Now with the ui-file design can that relatively easy changed. Perhaps after a first "field test"?</li>
</ul>

<ul class="remarkup-list">
<li class="remarkup-list-item">Your screenshot looks indeed not so nice with the big Document column and the small Session one. Updated as suggested and renamed to a shorter text. How about to remove (hide) the Document/Files column? They show by the way not the correct count when you open/closed some in the meantime. I have an additionally patch here which try to solve this but looks to me a little overkill, but perhaps you like it.</li>
</ul>

<p><a href="https://phabricator.kde.org/F6440381" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6440381: 1543127536.png</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R40 Kate</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D16926?vs=45709&id=46175">https://phabricator.kde.org/D16926?vs=45709&id=46175</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D16926">https://phabricator.kde.org/D16926</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kate/CMakeLists.txt<br />
kate/data/kateui.rc<br />
kate/katemainwindow.cpp<br />
kate/session/katesessionchooser.cpp<br />
kate/session/katesessionchooser.h<br />
kate/session/katesessionmanagedialog.cpp<br />
kate/session/katesessionmanagedialog.h<br />
kate/session/katesessionmanagedialog.ui<br />
kate/session/katesessionmanager.cpp<br />
kate/session/katesessionmanager.h<br />
kate/session/katesessionopendialog.cpp<br />
kate/session/katesessionopendialog.h</div></div></div><br /><div><strong>To: </strong>loh.tar, Kate<br /><strong>Cc: </strong>anthonyfieroni, cullmann, kwrite-devel, Kate, michaelh, ngraham, demsking, sars, dhaumann<br /></div>