<table><tr><td style="">ndavis 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/D22452">View Revision</a></tr></table><br /><div><div><p>The latest version of this patch is better because it works well for people using the Breeze GTK theme with the Breeze colorscheme and it doesn't make the Breeze Dark GTK theme worse when combined with the Breeze Dark colorscheme. However, it does cause Breeze GTK to look odd when used with the Breeze Dark colorscheme: <a href="https://phabricator.kde.org/F7027817" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F7027817: Screenshot_20190715_124709.png</a></p>
<p>Normally, I wouldn't consider this too much of a problem since people usually want to use the same theme system wide, but sometimes apps that use GTK themes don't play well with dark GTK themes. It is realistic to assume some people who normally use Breeze Dark will use the Breeze GTK theme for GTK apps via the system settings rather than an environment variable for the few problematic apps (<tt style="background: #ebebeb; font-size: 13px;">GTK_THEME=Breeze app-name-here</tt>). Examples of apps that don't play well with dark GTK themes are Eclipse, <a href="https://phabricator.kde.org/file/data/7x4wgf3q3pawfkdkakjf/PHID-FILE-vy5cmecnuljriju5ve2d/Screenshot_20190715_130024.png" class="remarkup-link" target="_blank" rel="noreferrer">Intellij IDEA</a> and Firefox (before version 68).</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/D22452">https://phabricator.kde.org/D22452</a></div></div><br /><div><strong>To: </strong>mthw, VDG, Breeze, ndavis, ngraham<br /><strong>Cc: </strong>cblack, ndavis, zzag, plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>