<table><tr><td style="">ngraham created this revision.<br />ngraham added a project: Okular.
</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/D7793" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Changed the colors used from the active theme so that there is never a white-on-light-blue situation that makes the text unreadable.</p>

<p>Here are the changes when using the Breeze color scheme:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">When an inactive tab is hovered over, the text is now dark (was light before)</li>
<li class="remarkup-list-item">When a selected tab loses focus, the text is now dark (was light before)</li>
</ul>

<p>Tested to make sure dark themes do not experience a regression.</p>

<p>BUG: 382139</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Tested on KDE Neon (dev unstable). Verified that with the default Breeze color scheme, tab text is more readable when hovered over and then active but not in focus. See before and after:</p>

<p><a href="https://phabricator.kde.org/F3909143" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">F3909143: After.png</a></p>

<p><a href="https://phabricator.kde.org/F3909142" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">F3909142: Before.png</a></p>

<p>Also tested these changes with all standard color schemes shipped with Neon to make sure that there were no visual regressions.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7793" rel="noreferrer">https://phabricator.kde.org/D7793</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>ui/sidebar.cpp</div></div></div><br /><div><strong>To: </strong>ngraham, aacid, Okular, KDE Applications, broulik, sander<br /><strong>Cc: </strong>aacid<br /></div>