<table><tr><td style="">fabianr 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><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D16926#366355" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D16926#366355</a>, <a href="https://phabricator.kde.org/p/loh.tar/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@loh.tar</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><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 Close button should probably be renamed "Select" or "Choose" so it's clear that the selected session will be made the active one</p></blockquote>

<p>The Close button close the dialog, like Abort. If you mean to rename "Open" I can do it. But IIRC was the old dialog buttons named the way as they are currently.</p></div>
</blockquote>

<p>I would be in favor off renaming the "Open ..." button to "Select", "Choose" or "Open Session" like it used to be. "Open ..." is kind of a well known button, where the user expects a file picker to open up,  and since "Open ..." does not require any additional user actions the ellipsis is against HIG the <a href="https://hig.kde.org/style/writing/labels.html" class="remarkup-link" target="_blank" rel="noreferrer">https://hig.kde.org/style/writing/labels.html</a> .</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><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 Don't ask again checkbox. What's being asked?</p></blockquote>

<p>It's borrowed from the removed (old) KateSessionChooser dialog and only shown in "Chooser Mode". Guess it was there named "Always use this choice" but that text was too long and would break the nice layout. I didn't find a better place for that. How about at tool tip text as hint?</p></blockquote>

<p>To me, in the old layout this was much clearer. The checkbox to the left of buttons to remember an action is a pattern that is used commonly in KDE, but I have not seen it below some buttons.</p>

<p>If you don't expect users to have a high number of sessions, maybe replacing the "Duplicate...", "Rename..." and "Delete ..." button with three icons in each row? That way a user has to only click once, instead of clicking twice (select row + button), you would get rid of the three buttons and the interaction pattern is probably easier to understand for the user. It is very obvious on which session the action is executed.</p>

<p>Disadvantage; if you have a lot of sessions, the icons might clutter the interface and while the trash can and the edit icon is probably obvious to anybody, the icon for "Duplicate ..." might not.</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/D16926">https://phabricator.kde.org/D16926</a></div></div><br /><div><strong>To: </strong>loh.tar, Kate, dhaumann, ngraham, VDG<br /><strong>Cc: </strong>fabianr, anthonyfieroni, cullmann, kwrite-devel, Kate, hase, michaelh, ngraham, demsking, sars, dhaumann<br /></div>