<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 7019ef7ca860b2922ef2f1701cd6a5d4a0fedc7e by Aaron Seigo to branch master.</pre>
 <br />









<p>- Commit</p>


<br />
<p>On March 11th, 2013, 6:33 p.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 11, 2013, 6:33 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;">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>