<table><tr><td style="">sandsmark created this revision.<br />sandsmark added reviewers: Konsole, hindenburg, tcanabrava.<br />sandsmark added a project: Konsole.<br />sandsmark requested review of this revision.
</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/D19482">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>The "standard"/fallback Fusion style for Qt uses QStyleHelper::backgroundColor() to determine if the scrollbar is drawn on has a dark background. QStyleHelper::backgroundColor() handles scrollbars differently from everything else, and tries to get the palette of the parent widget, and then gets the QPalette::Base color of that (that probably makes sense for someone other than me).</p>

<p>I think the scrollbar in the Qt Fusion style has way too low contrast in the first place, but this way we should at least invoke the right code in qfusionstyle, so it can't blame konsole for it.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Break with gdb at qfusionstyle.cpp:2539, verify that isDarkBg is true.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D19482">https://phabricator.kde.org/D19482</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/TerminalDisplay.cpp</div></div></div><br /><div><strong>To: </strong>sandsmark, Konsole, hindenburg, tcanabrava<br /><strong>Cc: </strong>konsole-devel, Konsole, gennad, maciejn, thsurrel, ngraham, maximilianocuria, hindenburg<br /></div>