<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/126968/">https://git.reviewboard.kde.org/r/126968/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On February 3rd, 2016, 12:18 a.m. UTC, <b>Xuetian Weng</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I just wonder, does this change also fix: https://bugs.kde.org/show_bug.cgi?id=241557 ?</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Also, I have impression that if there's a big difference between the size of two tooltip window, it will look bad.</p></pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">No, completely unrelated issue.</p></pre>
<br />
<p>- Thomas</p>
<br />
<p>On February 2nd, 2016, 8:51 p.m. UTC, Marco Martin wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for kwin and Plasma.</div>
<div>By Marco Martin.</div>
<p style="color: grey;"><i>Updated Feb. 2, 2016, 8:51 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="https://bugs.kde.org/show_bug.cgi?id=347863">347863</a>
</div>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kwin
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">this effect, derived from the Maximize one, will take the place of the manual window position animation that Plasma tooltip are using.
this should cause less problems as animationg positions on X is very error prone, plus it's less jarring when the tooltip sizes changes too, since that gets animated as well (behavior similar to Window7 taskbar tooltips)
look:
https://www.youtube.com/watch?v=sxE23ZgkkpU
slow motion:
https://www.youtube.com/watch?v=jDByfncO568</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">replaces https://git.reviewboard.kde.org/r/126870</p></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>effects/CMakeLists.txt <span style="color: grey">(dec50a9)</span></li>
<li>effects/backgroundcontrast/contrast.cpp <span style="color: grey">(168deb0)</span></li>
<li>effects/blur/blur.cpp <span style="color: grey">(a360f03)</span></li>
<li>effects/morphingpopups/CMakeLists.txt <span style="color: grey">(PRE-CREATION)</span></li>
<li>effects/morphingpopups/package/CMakeLists.txt <span style="color: grey">(PRE-CREATION)</span></li>
<li>effects/morphingpopups/package/contents/code/morphingpopups.js <span style="color: grey">(PRE-CREATION)</span></li>
<li>effects/morphingpopups/package/metadata.desktop <span style="color: grey">(PRE-CREATION)</span></li>
<li>libkwineffects/kwinanimationeffect.h <span style="color: grey">(f59eedc)</span></li>
<li>libkwineffects/kwinanimationeffect.cpp <span style="color: grey">(579535b)</span></li>
<li>scripting/scriptedeffect.h <span style="color: grey">(2a77a2f)</span></li>
<li>scripting/scriptedeffect.cpp <span style="color: grey">(7ab065b)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/126968/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>