<table><tr><td style="">romangg added a subscriber: zzag.<br />romangg 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/D15907">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/D15907#417286" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D15907#417286</a>, <a href="https://phabricator.kde.org/p/zzag/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@zzag</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Alright, I'm done.</p></div>
</blockquote>

<p>Maybe you misunderstood the last comment. Your autotest does not test the case if the size of the button is negative. Since the new runtime code has different code paths for <tt style="background: #ebebeb; font-size: 13px;">d->geometry.width() < 0</tt> and <tt style="background: #ebebeb; font-size: 13px;">d->geometry.height() < 0</tt> this case should be tested. The current autotest does it. You could have added some more rows for that to your autotest proposal to do this as well.</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>I don't understand what's the point of review where comments are ignored. The fact that reviewer has to address review comments strikes me odd the most, like it's his/her/their responsibility. It's really sad that we don't care about simplicity of code.</p></blockquote>

<p>I won't discuss reviews in general with you right now. Take a look at my code contributions in the last few months that you commented on. How many of your comments did I react to and how many not? And if you don't get why I ignored some of your comments think again in the broad scope of the project and in particular in the context of the current discussion about this single autotest. Also stop using untrue passive-aggressive statements like "It's really sad that we don't care about simplicity of code." if you don't want to generate animosity.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R129 Window Decoration Library</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15907">https://phabricator.kde.org/D15907</a></div></div><br /><div><strong>To: </strong>romangg, KWin, davidedmundson<br /><strong>Cc: </strong>zzag, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>