<table><tr><td style="">ohelin 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/D16365">View Revision</a></tr></table><br /><div><div><p>I checked the both the diff_part files next to each other. All the other differences were quite easy to see, but this one bugs me a little:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">scrollbar.vertical button.down {
  -gtk-icon-source: -gtk-icontheme("pan-down-symbolic"); }
scrollbar.vertical button.up {
  -gtk-icon-source: -gtk-icontheme("pan-up-symbolic"); }
scrollbar.horizontal button.down {
  -gtk-icon-source: -gtk-icontheme("pan-end-symbolic"); }
scrollbar.horizontal button.up {
  -gtk-icon-source: -gtk-icontheme("pan-start-symbolic"); }</pre></div>

<p>So that was in the old gtk-dark theme, but not in the new one. However, it's probably just some obsolete stuff anyway. From the few differences between the theme files, I think most are just leftovers from the past and simply forgotten to update. That alone speaks for the usefulness of this patch, as probably there shouldn't be any differences in margins for example between a light and a dark version of a theme.</p>

<p>So, for the code part I'd say it's fine, unless someone knows better. I don't have any visual experience using Breeze, so judging from the quick look I guess it's fine, too.</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/D16365">https://phabricator.kde.org/D16365</a></div></div><br /><div><strong>To: </strong>ohelin<br /><strong>Cc: </strong>ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>