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










<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On October 27th, 2014, 3:44 a.m. IST, <b>David Edmundson</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  


<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
 <thead>
  <tr>
   <th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
    <a href="https://git.reviewboard.kde.org/r/120806/diff/1/?file=322089#file322089line2" style="color: black; font-weight: bold; text-decoration: underline;">applets/fifteenPuzzle/package/contents/config/config.qml</a>
    <span style="font-weight: normal;">

     (Diff revision 1)

    </span>
   </th>
  </tr>
 </thead>



 
 

 <tbody>

  <tr>
    <th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
    <th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">2</font></th>
    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="cm"> * Copyright 2013  Bhushan Shah <bhush94@gmail.com></span></pre></td>
  </tr>

 </tbody>

</table>

  <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;">This guy gets everywhere.</p></pre>
 </blockquote>



 <p>On October 27th, 2014, 9:34 p.m. IST, <b>Jeremy Whiting</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;">Yep, I copied config.qml from fuzzyclock, so left the copyrights from there.</p></pre>
 </blockquote>







</blockquote>
<pre style="margin-left: 1em; 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;">o.O</p></pre>
<br />




<p>- Bhushan</p>


<br />
<p>On October 27th, 2014, 9:35 p.m. IST, Jeremy Whiting 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 Plasma and David Edmundson.</div>
<div>By Jeremy Whiting.</div>


<p style="color: grey;"><i>Updated Oct. 27, 2014, 9:35 p.m.</i></p>









<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
kdeplasma-addons
</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;">Config options from previous c++ version kept though names are a bit diferent.
Images not yet supported.
Puzzle starts at 0 in the top left corner, maybe should start with 1 in the corner though.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This is my first port to plasma qml, so I may have used some of the non suggested/recommended components, if so let me know and I'll fix it.</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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I've tested it with plasmoidviewer -a org.kde.plasma.fifteenpuzzle. </p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The icon is not working in the widget adder, not sure where to install that to for it to work.
Images not supported yet, though they are in the config, maybe should remove from config until they are supported?</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>applets/fifteenPuzzle/src/fifteenPuzzleConfig.ui <span style="color: grey">(ff82f331db4cee2d66f526954be63f0f5d81d250)</span></li>

 <li>applets/fifteenPuzzle/src/fifteenPuzzle.cpp <span style="color: grey">(8a1528988f3e693d20179db4a209309b0aad87fd)</span></li>

 <li>applets/CMakeLists.txt <span style="color: grey">(63e6e25628d18ee474231acd2a21711841dee592)</span></li>

 <li>applets/fifteenPuzzle/CMakeLists.txt <span style="color: grey">(04d5e55fd246684855d49484f1233dac054a0124)</span></li>

 <li>applets/fifteenPuzzle/Messages.sh <span style="color: grey">(PRE-CREATION)</span></li>

 <li>applets/fifteenPuzzle/icons/CMakeLists.txt <span style="color: grey">(106884f432c1d1e0b0584959af854c79ede4ea6d)</span></li>

 <li>applets/fifteenPuzzle/icons/hisc-app-fifteenpuzzle.svgz <span style="color: grey">()</span></li>

 <li>applets/fifteenPuzzle/images/blanksquare.svg <span style="color: grey">()</span></li>

 <li>applets/fifteenPuzzle/package/contents/config/config.qml <span style="color: grey">(PRE-CREATION)</span></li>

 <li>applets/fifteenPuzzle/package/contents/config/main.xml <span style="color: grey">(PRE-CREATION)</span></li>

 <li>applets/systemloadviewer/package/contents/ui/ColorPicker.qml <span style="color: grey">(92062db546dcff67f930d4888180f4e753798c27)</span></li>

 <li>applets/fifteenPuzzle/src/piece.h <span style="color: grey">(d0e58d0f9d38d4a1ef2110b974b3f4f6938293e1)</span></li>

 <li>applets/fifteenPuzzle/src/piece.cpp <span style="color: grey">(2efb72ecf69d9beaa53367bc2f3c9cee88238f28)</span></li>

 <li>applets/fifteenPuzzle/src/fifteenPuzzle.h <span style="color: grey">(cf7885380f0e152d51cf2dc7557444e9b425b596)</span></li>

 <li>applets/fifteenPuzzle/package/contents/ui/configAppearance.qml <span style="color: grey">(PRE-CREATION)</span></li>

 <li>applets/fifteenPuzzle/package/contents/ui/main.qml <span style="color: grey">(PRE-CREATION)</span></li>

 <li>applets/fifteenPuzzle/plasma-applet-fifteenPuzzle.desktop <span style="color: grey">(513cc0084df7247a520807620361b0426623727e)</span></li>

 <li>applets/fifteenPuzzle/src/Messages.sh <span style="color: grey">(bab24ae73049f37d9693cf062eaaa98ca1e6bab0)</span></li>

 <li>applets/fifteenPuzzle/src/fifteen.h <span style="color: grey">(2a27f5b109988003de45fb64c457484ebdfdbc8b)</span></li>

 <li>applets/fifteenPuzzle/src/fifteen.cpp <span style="color: grey">(ebdcf2c0756a17ea174c0fc5fd106e157b223063)</span></li>

 <li>applets/fifteenPuzzle/package/contents/ui/ColorPicker.qml <span style="color: grey">(PRE-CREATION)</span></li>

 <li>applets/fifteenPuzzle/package/contents/ui/FifteenPuzzle.qml <span style="color: grey">(PRE-CREATION)</span></li>

 <li>applets/fifteenPuzzle/package/contents/ui/Piece.qml <span style="color: grey">(PRE-CREATION)</span></li>

</ul>

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






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








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