<table><tr><td style="">dhaumann 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/D4329" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>First of all a general note: Unfortunately, we have had quite some issues with getting the geometry right in KMessageWidget over the past few years. Essentially, the problem is that KMessageWidgets animates its show and hide events, and therefore caches its initial and its final geometry, as well as the pixmap to render to avoid stretching the KMessageWidget's contents. That said, doing this caching too early results in a final geometry that is wrong (we had this issue in Kate a lot).</p>

<p>This may sound a bit off-topic, but the point is: it's easy to break things. If you in addition also run the unit test 'messagetest' of KTextEditor and if it passes, then I'm fine with this change. It looks reasonable after all :-)</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/D4329" rel="noreferrer">https://phabricator.kde.org/D4329</a></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>kossebau, Frameworks, dhaumann<br /><strong>Cc: </strong>cfeck<br /></div>