<table><tr><td style="">nhiga 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/D21979">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/D21979#484281" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D21979#484281</a>, <a href="https://phabricator.kde.org/p/ngraham/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ngraham</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Sorry, my mistake. I missed that this tooltip is set on the wrong checkbox. It needs to be set on the "CPUs separately" checkbox</p></div>
</blockquote>

<p>This is intentional - if we set this on the "CPUs separately" checkbox, <a href="https://bugreports.qt.io/browse/QTBUG-30801" class="remarkup-link" target="_blank" rel="noreferrer">QTBUG-30801</a> will make this patch not useful, because the tooltip will not be shown when the checkbox is disabled (i.e. grayed out), and therefore Jung's issue will not be addressed (see below).</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>In <a href="https://bugs.kde.org/show_bug.cgi?id=408959" class="remarkup-link" target="_blank" rel="noreferrer">Bug 408959</a>, Jung wrote:<br />
The "CPUs separately" box in the system load viewer widget configuration is grayed out. There is no indication what I could/should do to be allowed to enable that checkbox.  That's at best confusing and at worst frustrating UI design.</p></blockquote>

<p>Another option would be to put this patch on hold and wait for QTBUG-30801 to be fixed, and find other ways to let the user know that he/she should choose "Compact bar" to enable "CPUs separately".</p></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D21979">https://phabricator.kde.org/D21979</a></div></div><br /><div><strong>To: </strong>nhiga, Plasma, VDG, ngraham<br /><strong>Cc: </strong>ngraham, plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>