<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/109433/">http://git.reviewboard.kde.org/r/109433/</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;">This review has been submitted with commit b6faef2fac5e0e715fdc1018c6a0eb7d859bff73 by Aaron Seigo to branch KDE/4.10.</pre>
<br />
<p>- Commit</p>
<br />
<p>On March 12th, 2013, 10:29 a.m. UTC, Aaron J. Seigo 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 Mark Gaiser.</div>
<div>By Aaron J. Seigo.</div>
<p style="color: grey;"><i>Updated March 12, 2013, 10:29 a.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;">Fixes zero size pager in default panel, makes it optional via a non-GUI config option which is set in the default panel layout script.
One thing that still needs work with this is that existing installations will suddenly have pagers that show up with 1 virtual desktop. An update script needs to be written for this before it can be pushed.</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;">None yet; just wanted to sketch out the proposed solution.</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>plasma/desktop/applets/pager/pager.h <span style="color: grey">(764cb01)</span></li>
<li>plasma/desktop/applets/pager/pager.cpp <span style="color: grey">(228af8e)</span></li>
<li>plasma/desktop/shell/data/layouts/org.kde.plasma-desktop.defaultPanel/contents/layout.js <span style="color: grey">(afd1f2c)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/109433/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>