<table><tr><td style="">ahmadsamir 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/D24672">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/D24672#547744" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D24672#547744</a>, <a href="https://phabricator.kde.org/p/mkoller/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@mkoller</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>why ?<br />
 I don't like lots of these changes.</p></div>
</blockquote>

<p>Because it doesn't follow the KDE coding style, AFAICS (and changing it to use that common style will make it easier for whoever is going to work on porting it away from kdelibs4support[1]).</p>

<p>Anyway, if you disagree, I'll drop this review request. :)</p>

<p>[1]<a href="https://phabricator.kde.org/T11576" class="remarkup-link" target="_blank" rel="noreferrer">https://phabricator.kde.org/T11576</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R374 KolourPaint</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D24672">https://phabricator.kde.org/D24672</a></div></div><br /><div><strong>To: </strong>ahmadsamir, kde-frameworks-devel, mkoller<br /><strong>Cc: </strong>ognarb, kde-frameworks-devel<br /></div>