<table><tr><td style="">mart 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/D3131" rel="noreferrer">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/D3131#59202" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D3131#59202</a>, <a href="https://phabricator.kde.org/p/hpereiradacosta/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@hpereiradacosta</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Hi Marco,</p>

<p>Couple of comments on the look of it</p>

<p>1/ if you set the animation time to somehting long (500ms), then hover on a scrollbar, in the groove, and not on the handle, while making sure that the view in which you hove does not have focus, in order to make sure that the handle is painted grey, not blue (either focus blue or hover blue), there is a glitch at the end of the animation where a somewhat too dark grey jumps to a lighter one. Must be some problem with transparency. Can you confirm, double check ? I 'think' this is also the case when you mouse-over the handle but this is less clear.</p></div>
</blockquote>

<p>yes, can reproduce, looking into that</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>2/ I wonder whether the unhovered scrollbar of a focused view should use the focus blue for the handle (as it is now, without your change), instead of light grey. I at least was using is somewhat strong blue to easily catch the view that have focus, instead of the much more subtle blue outline. What do you think ?</p></blockquote>

<p>for me either way is fine, would like also vdg opinion on it</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>3/ I agree that the animation looks good on views like dolphin and systemsettings. However for views with a header, or when you have both vertical and horizontal bars it looks stranger . But this is a matter of taste</p></blockquote>

<p>yeah, asked about this concern back them to vdg, they say basically that yeah, it's not that pretty, but better than nothing</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>4/ finally, I had this crazy idea :)<br />
 What if ... instead of making the groove "fade-in" on hover, we would make it "expand" vertically, from around the handle ? as if it would appear from under the handle and grow to ocupy the full height of the view ? Do you think it could look cool ?</p></blockquote>

<p>may be cool.. (or also look a bit too jumpy) one thing, since i'm writing a qstyle-based theme for QtQuickControls2, this would make it harder to replicateon that weird qml/qstyle hybrid</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rBREEZE Breeze</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D3131" rel="noreferrer">https://phabricator.kde.org/D3131</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>mart, Plasma, Breeze, VDG, hpereiradacosta<br /><strong>Cc: </strong>plasma-devel, abetts, lesliezhai, ali-mohamed, jensreuterberg, sebas<br /></div>