<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/D16169">View Revision</a></tr></table><br /><div><div><p>+1 for implementing the feature.</p>
<p>However, I'm always a bit wary about adding new off-by-default options. Making something optional in the first place is an a certain lack of confidence in the feature; having it off by default magnifies that. If this should really be both optional and off by default, should it really be done at all? My vote would be for turning it on by default, but I think ideally, this would be a global setting somewhere in System Settings (maybe the same place we'd put the options requested in <a href="https://bugs.kde.org/show_bug.cgi?id=376992" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=376992</a>) and would affect all KDE apps with tabs.</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/D16169">https://phabricator.kde.org/D16169</a></div></div><br /><div><strong>To: </strong>thsurrel, Kate, VDG<br /><strong>Cc: </strong>ngraham, kwrite-devel, michaelh, demsking, cullmann, sars, dhaumann<br /></div>