<table><tr><td style="">cullmann 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/D4657" rel="noreferrer">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/D4657#87433" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D4657#87433</a>, <a href="https://phabricator.kde.org/p/mwolff/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@mwolff</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Christoph, Dominik,: When comparing this implementation to the one in Kate:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">why exactly is the <tt style="background: #ebebeb; font-size: 13px;">BarState</tt> wrapper needed?</li>
<li class="remarkup-list-item">i.e. is <tt style="background: #ebebeb; font-size: 13px;">slotUpdateBottomViewBar</tt> really needed, isn't that handled in KTextEditor? If it's needed, shouldn't this be in KTextEditor to call <tt style="background: #ebebeb; font-size: 13px;">{show,hide}ViewBar</tt>?</li>
<li class="remarkup-list-item">Kate guards against <tt style="background: #ebebeb; font-size: 13px;">nullptr</tt> view bars, is this really needed? Put differently - when would this ever occur? Shouldn't this be handled by KTextEditor to not call <tt style="background: #ebebeb; font-size: 13px;">addWidgetToViewBar</tt>?</li>
</ul></div>
</blockquote>

<p>Hmm, for the null check, I guess that is not needed, more caution ;)<br />
For the other two things: Isn't the problem that KTextEditor can't keep track of view switches and has no concept of active view and the slotUpdateBottomViewBar() is used to restore the right view bar if the view is changed?<br />
But that code is old, I could be mistaken ;=) And the state var in the BarState is awful, it means "visible" or?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R33 KDevPlatform</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D4657" rel="noreferrer">https://phabricator.kde.org/D4657</a></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>mwolff, KDevelop, dhaumann, cullmann<br /><strong>Cc: </strong>kdevelop-devel, KDevelop, Pilzschaf, akshaydeo, surgenight, arrowdodger<br /></div>