<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/D25543">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D25543#568159" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D25543#568159</a>, <a href="https://phabricator.kde.org/p/hpereiradacosta/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@hpereiradacosta</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>You are missing copyright information and license in all the newly created files.</p></div>
</blockquote>

<p>Thanks, should I just copy/paste the info from breezestyle.cpp?</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>On the review side: it is impossible to actually review, right ?</p></blockquote>

<p>I was kind of worried that might be the case.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>As for the conceptual side: I fear this is addressing a non existing issue, and giving a wrong impression about how one should hack on a widget style. It is wrong to think that you can hack on a widget style widget by widget without consideration about how they should appear one with respect to the others, how they should align one with respect to the other, and how the other widgets are implemented. You do need to know the whole code and interplay before starting to hack anyway. The splitting does not change this. In the end it might just result in a lot of duplicated code.</p></blockquote>

<p>I have seen others in the VDG chat room say that the massive length of breezestyle.cpp made hacking on Breeze too intimidating. I understand your concern about giving the wrong impression though. I'm hoping to use these new files as a way of categorizing by visible widget types ("Where is the code for tabs? In tabbar.cpp", not exact Qt Widget classes. There's still nothing stopping others from reusing code from one widget category in another since all the functions are still defined in breezehelper.h and breezestyle.h.</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/D25543">https://phabricator.kde.org/D25543</a></div></div><br /><div><strong>To: </strong>ndavis, Plasma, Breeze<br /><strong>Cc: </strong>hpereiradacosta, ngraham, IlyaBizyaev, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>