<table><tr><td style="">yurchor 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/D16133">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/D16133#341434" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D16133#341434</a>, <a href="https://phabricator.kde.org/p/cfeck/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@cfeck</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Checking the previous statusbar code, the id's have a range of 1 ... 4, so there should be 4 sections at list index 0 ... 3, accessed by <tt style="background: #ebebeb; font-size: 13px;">at(id - 1)</tt>. The current patch adds 5 sections, and label at index 0 is never used.</p>

<p>I have not tested the patch, but the code to keep the status bar items at fixed sizes was removed, and I fear that hovering over the graphs makes the items all jump around due to sizes changing. Correct?</p></div>
</blockquote>

<p>Yes, it is correct. But there is no jumping. First labels are coordinates, they do not change much in size (I can record a screencast if you like to confirm). The other two labels are jumping in the same manner as in the unpatched version. The original behavior can be easily mimicked by adding several lines of ugly code to asses the width of 16 characters and a condition.</p>

<p>Yes, one label is empty all the time. The code is simpler, I guess that's what we call  "the progress" in the new version. Should it be changed to mimic the original by shifting index by one in all the files to remove this extra label?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R334 KmPlot</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D16133">https://phabricator.kde.org/D16133</a></div></div><br /><div><strong>To: </strong>yurchor, KDE Edu<br /><strong>Cc: </strong>cfeck, kde-edu, narvaez, apol<br /></div>