<table><tr><td style="">ngraham added reviewers: Breeze, ndavis.<br />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/D27938">View Revision</a></tr></table><br /><div><div><p>Wow, looks like this was a lot of work!</p>

<p>Can you add a bit more information about what your goal is here? You're adding different icon styles for the titlebar buttons that you can use while still using the Breeze window decoration theme? Before-and-after screenshots would help too. See also <a href="https://community.kde.org/Infrastructure/Phabricator#Include_some_screenshots" class="remarkup-link" target="_blank" rel="noreferrer">https://community.kde.org/Infrastructure/Phabricator#Include_some_screenshots</a></p>

<p>Could you add a few words regarding why you think adding this option to the Breeze window decoration theme is right right place for this, as opposed to creating a new window decoration theme or just changing the default appearance? If there's something wrong with the current button appearance, maybe we should just change it. If not, and this is just to satisfy people with different aesthetic preferences, why not make a different window decoration theme?</p></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/D27938">https://phabricator.kde.org/D27938</a></div></div><br /><div><strong>To: </strong>paulm, hpereiradacosta, Breeze, ndavis<br /><strong>Cc: </strong>ngraham, plasma-devel, manueljlin, Orage, LeGast00n, The-Feren-OS-Dev, cblack, konkinartem, ian, jguidon, Ghost6, jraleigh, zachus, MrPepe, fbampaloukas, squeakypancakes, alexde, IohannesPetros, GB_2, trickyricky26, ragreen, crozbo, ndavis, ZrenBot, firef, skadinna, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, aaronhoneycutt, abetts, sebas, apol, ahiemstra, mbohlender, mart<br /></div>