<table><tr><td style="">broulik created this revision.<br />broulik added reviewers: Plasma, VDG.<br />Restricted Application added projects: Plasma, Frameworks.<br />Restricted Application added subscribers: Frameworks, plasma-devel.<br />broulik requested review of this revision.
</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/D10077" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Unset the Label's default height which completely confuses the renderer either shifting the item vertically (e.g. when toggling system tray popup), or letting it get too wide (e.g. when moving from a two-line tooltip to a one-line tooltip).</p>

<p>BUG: 386260<br />
BUG: 389371</p></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Opened and closed System Tray popup, tooltip was correctly aligned now (bug 389371)</li>
</ul>

<p><a href="https://phabricator.kde.org/F5672178" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">F5672178: Screenshot_20180124_155427.png</a></p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Moved from NM to Battery plasmoid, tooltip was correctly sized now (bug 386260)</li>
</ul>

<ul class="remarkup-list">
<li class="remarkup-list-item">Moved from task manager tooltip to kickoff and vice-versa, was still correct</li>
<li class="remarkup-list-item">Observed all tooltips having a sane size.</li>
</ul>

<p>When subtext linebreaks the tooltip can appear too wide because the word wrapped onto the next line but this isn't easily fisable, not without using a separate <tt style="background: #ebebeb; font-size: 13px;">TextMetrics</tt> instance and other sledgehammer methods</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R242 Plasma Framework (Library)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10077" rel="noreferrer">https://phabricator.kde.org/D10077</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/declarativeimports/core/private/DefaultToolTip.qml</div></div></div><br /><div><strong>To: </strong>broulik, Plasma, VDG<br /><strong>Cc: </strong>plasma-devel, Frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>