<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/126388/">https://git.reviewboard.kde.org/r/126388/</a>
     </td>
    </tr>
   </table>
   <br />



<p>

Ship it!

</p>



 <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;">Looks like an excellent addition, this will really improve info fme.users.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I have reviewed the diffs and it looks good</p></pre>
 <br />









<p>- Rick Timmis</p>


<br />
<p>On July 13th, 2016, 5:17 p.m. UTC, Carlo Vanini 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 Kubuntu and Muon Package Management Suite.</div>
<div>By Carlo Vanini.</div>


<p style="color: grey;"><i>Updated July 13, 2016, 5:17 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="http://bugs.kde.org/show_bug.cgi?id=332453">332453</a>


</div>



<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
muon
</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;">Add more columns to the list of packages and in the "preview changes" tab.
The additional columns are hidden by default and can be shown through the context menu by right-clicking on the list header.</p></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;"><ul style="padding: 0;text-rendering: inherit;margin: 0 0 0 1em;line-height: inherit;white-space: normal;">
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">start muon</li>
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">right-click on the list header</li>
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">a context menu appears, select/deselect columns to show</li>
</ul></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>src/CMakeLists.txt <span style="color: grey">(86205062cdc385fe8e65d3d22f46f040360d8277)</span></li>

 <li>src/ManagerWidget.h <span style="color: grey">(2daad860e9b89f16fce46862bde8b7d9d0d664c8)</span></li>

 <li>src/ManagerWidget.cpp <span style="color: grey">(05fef2d6891876c7a064030b353ab51d84bd551e)</span></li>

 <li>src/PackageModel/PackageDelegate.cpp <span style="color: grey">(f39cd78baf95b02350592554392484308f681244)</span></li>

 <li>src/PackageModel/PackageModel.h <span style="color: grey">(7a89845291ac3fdc7ad162dd9e3543b178853f7f)</span></li>

 <li>src/PackageModel/PackageModel.cpp <span style="color: grey">(9bde61500beec00e97ce5a675737ba0b4ef54714)</span></li>

 <li>src/PackageModel/PackageView.cpp <span style="color: grey">(3b1936679dd984a7f11c94baacabd31944562485)</span></li>

 <li>src/PackageModel/PackageViewHeader.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/PackageModel/PackageViewHeader.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/PackageModel/PackageWidget.h <span style="color: grey">(def2ef864790fcec5a02cddd5eeb79726ccd99ec)</span></li>

 <li>src/PackageModel/PackageWidget.cpp <span style="color: grey">(ea4307ba7a040f75c25c9b9b0cb06e2320a51385)</span></li>

 <li>src/config/muon.kcfg <span style="color: grey">(f880169b164b924a2957e8bf9256706750fe5082)</span></li>

</ul>

<p><a href="https://git.reviewboard.kde.org/r/126388/diff/" style="margin-left: 3em;">View Diff</a></p>






  </td>
 </tr>
</table>







  </div>
 </body>
</html>