<table><tr><td style="">drosca 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/D3813" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Currently, it tries to find QAbstractScrollArea in parent chain and use focused state of this scroll area (+ there is a special case for KTextView). There is no standard way to specify the widget that is being scrolled by the scrollbar. This only works with standard Qt widgets and breaks when you implement custom widget that uses scrollbars.</p>

<p>This proposed solution also checks if the scrollbar itself have the focus, which I think makes sense and should not have any side-effects. Even in the case when scrollbar is actually able to get focus (ie scrollbar having focus while the scrolled widget not having it) it still should be rendered as focused, because it *is* focused.</p>

<p>My use case is scrollbar controlling widget that is not parent of the scrollbar and scrollbar having focusproxy the scrolled widget.</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D3813#inline-15126" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">hpereiradacosta</span> wrote in <span style="color: #4b4d51; font-weight: bold;">breezestyle.cpp:3067</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">You sure this is actually needed ? The primitive is supposed to do an horizontal line below the passed rect. Never seen that happened for scrollbars. Does it really ? (without the patch)</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Yes, this is needed because it is painted when the scrollbar have focus (which is the case this patch is trying to support).</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R31 Breeze</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D3813" rel="noreferrer">https://phabricator.kde.org/D3813</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>drosca, Plasma, hpereiradacosta<br /><strong>Cc: </strong>plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas<br /></div>