<table><tr><td style="">cfeck requested changes to this revision.<br />cfeck added a comment.<br />This revision now requires changes to proceed.
</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/D7010" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>The title says "when text changes", but you call it even if the text does not change. Note that squeezeTextToLabel() is called from many places, including resizeEvent(), so avoid calling updateGeometry() if not needed.</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/D7010" rel="noreferrer">https://phabricator.kde.org/D7010</a></div></div><br /><div><strong>To: </strong>brauch, cfeck<br /><strong>Cc: </strong>aacid, Frameworks<br /></div>