<table><tr><td style="">ngraham 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/D17814">View Revision</a></tr></table><br /><div><div><p>For now the default can remain none, but I recommend we change the default to show a close button--if not in this patch, then in another one.</p>

<p>We are currently working on harmonizing the appearance and behavior of tabs throughout KDE software (<a href="https://phabricator.kde.org/T10233" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">T10233</a>) and closable tabs needs close buttons. It's one of the ways that the tab communicates to the user that it's closable. Other KDE apps with closable tabs (Dolphin, Kate, KDevelop, Okteta, Okular) all have close buttons shown on their tabs. Konsole should too.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17814">https://phabricator.kde.org/D17814</a></div></div><br /><div><strong>To: </strong>marssola, tcanabrava, VDG, Konsole, ngraham<br /><strong>Cc: </strong>rizzitello, abetts, ngraham, hindenburg, konsole-devel, thsurrel, maximilianocuria<br /></div>