<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/111618/">http://git.reviewboard.kde.org/r/111618/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I'm sorry, but that approach is not good enough. The problem with the small launcher items is that it throws the assumption out the door that all items are of the same width, which has far-ranging consequences throughout the entire layout code: Calculating how many items to show per row in multi-row layouts, calculating at which point the taskbar is considered "full" to control opportunistic grouping and row overflow, calculating the size hints to make opportunistically growing panels work correctly, and so on. It makes the layout problem non-deterministic.
This is also why the old taskbar is actually spectacularly broken on launcher items: Try some launchers together with a two-row layout and marvel at the fallout.
I'm still working on a proper solution.</pre>
<br />
<p>- Eike</p>
<br />
<p>On July 20th, 2013, 2:58 p.m. UTC, Mark Gaiser wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Plasma and Eike Hein.</div>
<div>By Mark Gaiser.</div>
<p style="color: grey;"><i>Updated July 20, 2013, 2:58 p.m.</i></p>
<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;">As described in the bug report, the launcher regressed with the move to the QML based tasks. I did my best to correct the launcher regression. The attachment shows how the launchers look with this patch.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </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;">Tested it and works just like it used to work in KDE 4.10.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=321128">321128</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>plasma/desktop/applets/tasks/package/contents/code/layout.js <span style="color: grey">(cb7a1c2)</span></li>
<li>plasma/desktop/applets/tasks/package/contents/ui/Task.qml <span style="color: grey">(70267fb)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/111618/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">File Attachments </h1>
<ul>
<li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/07/20/fix_launcher.png">fix_launcher.png</a></li>
</ul>
</td>
</tr>
</table>
</div>
</body>
</html>