<table><tr><td style="">ngraham 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/D21639">View Revision</a></tr></table><br /><div><div><p>Ah, this change only affects those horrible scrollbars that disappear when not being used and leave you unable to see at a glance what position you're at in the view or even that the view is scrollable in the first place. This change seems to work fine and improves the UI for that particular use case, so +1.</p>

<p>However Phab doesn't expose that this patch has significant formatting issues. It looks like your editor changed all the line endings for <tt style="background: #ebebeb; font-size: 13px;">src/gtk320/widgets/_scrollbar.scss</tt>, which is an undesired change that must be reverted. There's also a hidden whitespace issue with the change in <tt style="background: #ebebeb; font-size: 13px;">src/gtk318/widgets/_scrollbar.scss</tt>. It helps to do a final <tt style="background: #ebebeb; font-size: 13px;">git diff</tt> before submitting the patch, which can help catch issues like this.</p>

<p>Please fix those issues, then we can land this.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R98 Breeze for Gtk</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D21639">https://phabricator.kde.org/D21639</a></div></div><br /><div><strong>To: </strong>mthw, VDG, Breeze, ndavis, ngraham<br /><strong>Cc: </strong>ngraham, ndavis, plasma-devel, LeGast00n, ericadams, jraleigh, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>