davidre added inline comments.

INLINE COMMENTS

> breezetoolsareamanager.cpp:109
> +                            window->setContentsMargins(0,1,0,0);
> +                        }
> +                    });

Maybe the decision whether to do this could only be made based on the colors of 
the active state so it's consistent with the inactive one.
Also this happens because the separator line? Do we actually need it? Currently 
there is no separator between titlebar and window and/or between toolbar and 
other window content?

REPOSITORY
  R31 Breeze

REVISION DETAIL
  https://phabricator.kde.org/D27669

To: cblack, #plasma, #breeze, #vdg
Cc: davidre, davidedmundson, hpereiradacosta, gvgeo, ngraham, manueljlin, 
niccolove, ndavis, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

Reply via email to