<table><tr><td style="">rkflx 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/D7977" rel="noreferrer">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/D7977#148689" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D7977#148689</a>, <a href="https://phabricator.kde.org/p/cfeck/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@cfeck</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Qt does not send a resize event</p></blockquote>

<p>But does it send QEvent::ContentsRectChange?</p></div>
</blockquote>

<p>No, because this one is only emitted by <tt style="background: #ebebeb; font-size: 13px;">setContentsMargin</tt>, but not by <tt style="background: #ebebeb; font-size: 13px;">setMargin</tt> (read <a href="https://phabricator.kde.org/D7164#143540" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;" rel="noreferrer">D7164#143540</a> (¹) again for what's the difference) or <tt style="background: #ebebeb; font-size: 13px;">setIndent</tt>.</p>

<p>Even if there were resize related events (in some situations we even get some), we are not interested in them. The label should react by squeezing, not by adjusting size, as the detailed investigations in <a href="https://phabricator.kde.org/T6982#111367" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">T6982#111367</a> conclude.</p>

<p><tt style="background: #ebebeb; font-size: 13px;">testChrome</tt> suggests to unify the behaviour of all its three chrome related calls. My patch is the most straightforward way to get there and I don't see how we could bolt on something convoluted through the event system without even reimplementing the setters to emit an otherwise missing event.</p>

<p>Let me know if you have any concerns against my approach, I'll gladly respond to them.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R236 KWidgetsAddons</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7977" rel="noreferrer">https://phabricator.kde.org/D7977</a></div></div><br /><div><strong>To: </strong>rkflx, Frameworks, dhaumann, cfeck<br /></div>