

<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/126793/">https://git.reviewboard.kde.org/r/126793/</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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Nice to see this applet coming back!</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I was wondering if there could be more code shared between this applet and our image wallpaper, perhaps even "just" allow the image wallpaper to function also as an applet.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">You have a lot of empty lines with whitespace in your code. Please check your editor settings.</p></pre>

 <br />


<div>


<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/126793/diff/1/?file=434177#file434177line19" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/code/utility.js</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">19</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="kd">function</span> <span class="nx">loop</span><span class="p">(</span><span class="nx">object</span><span class="p">,</span><span class="nx">callback</span><span class="p">)</span> <span class="p">{</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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 function seems unused</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434177#file434177line29" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/code/utility.js</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">29</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="kd">function</span> <span class="nx">remap</span><span class="p">(</span> <span class="nx">old_value</span><span class="p">,</span> <span class="nx">old_min</span><span class="p">,</span> <span class="nx">old_max</span><span class="p">,</span> <span class="nx">new_min</span><span class="p">,</span> <span class="nx">new_max</span> <span class="p">)</span> <span class="p">{</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">Also given this one is used only once I wonder if the utility.js is needed at all.</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434178#file434178line31" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/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">31</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">         <span class="k">icon:</span> <span class="s2">"folder-new"</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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 is an action icon and shouldn't be used for this prupose</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434179#file434179line9" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/config/main.xml</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">9</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="nt"><default></span>6850<span class="nt"></default></span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">Where does this come from?</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434180#file434180line34" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/ui/ConfigGeneral.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">34</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="nx">property</span> <span class="kr">int</span> <span class="k">cfg_interval:</span> <span class="mi">6850</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">Can't this also be an alias property? At least try to avoid having the default value hardcoded in two places</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434180#file434180line55" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/ui/ConfigGeneral.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">55</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">            <span class="nx">TextField</span> <span class="p">{</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">I would suggest using a SpinBox here, it also supports appending a suffix</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434180#file434180line69" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/ui/ConfigGeneral.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">69</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">            <span class="nx">Text</span> <span class="p">{</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">Use Label from QtQuick.Controls instead of QtQuick Text</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434180#file434180line80" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/ui/ConfigGeneral.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">80</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">                <span class="k">checked:</span> <span class="kc">true</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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 shouldn't be neccessary.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Plasma populates all cfg_foo properties with the stored value or the default if none is set.</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434180#file434180line85" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/ui/ConfigGeneral.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">85</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="nx">RowLayout</span> <span class="p">{</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">These RowLayouts around the actual control seem superfluous</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434181#file434181line35" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/ui/ConfigPaths.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">35</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="nx">property</span> <span class="nx">string</span> <span class="k">cfg_pathList:</span> <span class="s2">""</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">pathList should be a StringList which will result in a JS Array, then you don't need all of this JSON.parse stuff.</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434182#file434182line47" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/ui/main.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">47</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="nx">property</span> <span class="kr">int</span> <span class="k">implicitWidth:</span> <span class="nx">units</span><span class="p">.</span><span class="nx">gridUnit</span> <span class="o">*</span> <span class="mi">30</span></pre></td>

  </tr>


  <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">48</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="nx">property</span> <span class="kr">int</span> <span class="k">implicitHeight:</span> <span class="nx">units</span><span class="p">.</span><span class="nx">gridUnit</span> <span class="o">*</span> <span class="mi">20</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">Item already has implicitWidth/implicitHeight, no need to define them explicitly</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434182#file434182line55" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/ui/main.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">55</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="k">Plasmoid.backgroundHints:</span> <span class="nx">config</span><span class="p">.</span><span class="nx">useBackground</span> <span class="o">?</span> <span class="mi">1</span> <span class="o">:</span> <span class="s2">"NoBackground"</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">PlasmaCore.Types.NoBackground, I think, string enums are deprecated</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434182#file434182line57" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/ui/main.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">57</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="nx">property</span> <span class="kr">int</span> <span class="k">minimumWidth:</span> <span class="nx">theme</span><span class="p">.</span><span class="nx">mSize</span><span class="p">(</span><span class="nx">theme</span><span class="p">.</span><span class="nx">defaultFont</span><span class="p">).</span><span class="nx">width</span> <span class="o">*</span> <span class="mi">45</span></pre></td>

  </tr>


  <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">58</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="nx">property</span> <span class="kr">int</span> <span class="k">minimumHeight:</span> <span class="nx">theme</span><span class="p">.</span><span class="nx">mSize</span><span class="p">(</span><span class="nx">theme</span><span class="p">.</span><span class="nx">defaultFont</span><span class="p">).</span><span class="nx">height</span> <span class="o">*</span> <span class="mi">18</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">Layout.minimumWidth / height ?</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434182#file434182line63" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/ui/main.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">63</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="nx">property</span> <span class="kd">var</span> <span class="k">config:</span> <span class="p">{}</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">You can just access plasmoid.configuration, don't store this in a separate object as then it won't update when the config changes.</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434182#file434182line65" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/ui/main.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">65</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="nx">property</span> <span class="nx">string</span> <span class="k">activeSource:</span> <span class="s2">""</span></pre></td>

  </tr>


  <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">66</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="nx">property</span> <span class="nx">string</span> <span class="k">transitionSource:</span> <span class="s2">""</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">I was wondering if, instead of storing the source, you have two Item properties holding the first and other image and then you can just swap the items.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Perhaps have a look at [plasma-workspace]/wallpapers/image/imagepackage/contents/ui/main.qml</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434182#file434182line73" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/ui/main.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">73</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="nx">property</span> <span class="kr">int</span> <span class="k">itemCount:</span> <span class="p">(</span><span class="nx">items</span><span class="p">.</span><span class="nx">count</span> <span class="o">+</span> <span class="nx">future</span><span class="p">.</span><span class="nx">length</span><span class="p">)</span></pre></td>

  </tr>


  <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">74</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="nx">property</span> <span class="nx">bool</span> <span class="k">hasItems:</span> <span class="p">((</span><span class="nx">itemCount</span> <span class="o">></span> <span class="mi">0</span><span class="p">)</span> <span class="o">||</span> <span class="p">(</span><span class="nx">future</span><span class="p">.</span><span class="nx">length</span> <span class="o">></span> <span class="mi">0</span><span class="p">))</span></pre></td>

  </tr>


  <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">75</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="nx">property</span> <span class="nx">bool</span> <span class="k">isTransitioning:</span> <span class="nx">faderAnimation</span><span class="p">.</span><span class="nx">running</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">These could be readonly property</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434182#file434182line230" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/ui/main.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">230</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="nx">Connections</span> <span class="p">{</span></pre></td>

  </tr>


  <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">231</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="k">target:</span> <span class="nx">plasmoid</span><span class="p">.</span><span class="nx">configuration</span></pre></td>

  </tr>


  <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">232</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>

  </tr>


  <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">233</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="k">onPathListChanged:</span> <span class="p">{</span></pre></td>

  </tr>


  <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">234</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">            <span class="nx">loadPathList</span><span class="p">()</span></pre></td>

  </tr>


  <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">235</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="p">}</span></pre></td>

  </tr>


  <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">236</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="ew">        </span></pre></td>

  </tr>


  <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">237</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="k">onIntervalChanged:</span> <span class="p">{</span></pre></td>

  </tr>


  <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">238</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">            <span class="nx">loadConfig</span><span class="p">()</span></pre></td>

  </tr>


  <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">239</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="p">}</span></pre></td>

  </tr>


  <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">240</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="ew">        </span></pre></td>

  </tr>


  <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">241</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="k">onRandomizeChanged:</span> <span class="p">{</span></pre></td>

  </tr>


  <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">242</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">            <span class="nx">loadConfig</span><span class="p">()</span></pre></td>

  </tr>


  <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">243</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="p">}</span></pre></td>

  </tr>


  <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">244</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="ew">        </span></pre></td>

  </tr>


  <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">245</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="k">onPauseOnMouseOverChanged:</span> <span class="p">{</span></pre></td>

  </tr>


  <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">246</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">            <span class="nx">loadConfig</span><span class="p">()</span></pre></td>

  </tr>


  <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">247</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="p">}</span></pre></td>

  </tr>


  <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">248</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="ew">        </span></pre></td>

  </tr>


  <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">249</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="k">onUseBackgroundChanged:</span> <span class="p">{</span></pre></td>

  </tr>


  <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">250</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">            <span class="nx">loadConfig</span><span class="p">()</span></pre></td>

  </tr>


  <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">251</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="p">}</span></pre></td>

  </tr>


  <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">252</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="ew">        </span></pre></td>

  </tr>


  <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">253</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="k">onLeftClickOpenImageChanged:</span> <span class="p">{</span></pre></td>

  </tr>


  <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">254</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">            <span class="nx">loadConfig</span><span class="p">()</span></pre></td>

  </tr>


  <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">255</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="p">}</span></pre></td>

  </tr>


  <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">256</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="ew">        </span></pre></td>

  </tr>


  <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">257</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="k">onShowCountdownChanged:</span> <span class="p">{</span></pre></td>

  </tr>


  <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">258</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">            <span class="nx">loadConfig</span><span class="p">()</span></pre></td>

  </tr>


  <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">259</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="p">}</span></pre></td>

  </tr>


  <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">260</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="p">}</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">All of this was unneccessary if you accessed plasmoid.configuration directly</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434182#file434182line267" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/ui/main.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">267</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="k">triggeredOnStart:</span> <span class="kc">false</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">The default value is false already</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434182#file434182line268" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/ui/main.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">268</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="k">onTriggered:</span> <span class="p">{</span> <span class="nx">nextItem</span><span class="p">()</span> <span class="p">}</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">The braces are not needed:</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">onTriggered: nextItem()</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434182#file434182line299" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/ui/main.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">299</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">                <span class="k">fillMode:</span> <span class="nx">Image</span><span class="p">.</span><span class="nx">PreserveAspectFit</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">I would like to be able to configure this</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434182#file434182line318" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/ui/main.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">318</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">                    <span class="k">onClicked:</span> <span class="p">{</span> <span class="k">if</span><span class="p">(</span><span class="nx">config</span><span class="p">.</span><span class="nx">leftClickOpenImage</span><span class="p">)</span> <span class="nx">Qt</span><span class="p">.</span><span class="nx">openUrlExternally</span><span class="p">(</span><span class="nx">activeSource</span><span class="p">)</span> <span class="p">}</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">onClicked: Qt.openUrlExternally(activeSource)

enabled: plasmoid.configuration.openImageOnLeftClick</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434182#file434182line325" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/ui/main.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">325</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="nx">DropShadow</span> <span class="p">{</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">I would put that into a Loader to avoid shader compilation if you're not actually using it</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434182#file434182line352" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/ui/main.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">352</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">            <span class="nx">NumberAnimation</span> <span class="p">{</span> <span class="k">target:</span> <span class="nx">frontImage</span><span class="p">;</span> <span class="k">property:</span> <span class="s2">"opacity"</span><span class="p">;</span> <span class="k">to:</span> <span class="mi">0</span><span class="p">;</span> <span class="k">duration:</span> <span class="mi">450</span> <span class="p">}</span></pre></td>

  </tr>


  <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">353</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">            <span class="nx">NumberAnimation</span> <span class="p">{</span> <span class="k">target:</span> <span class="nx">bufferImage</span><span class="p">;</span> <span class="k">property:</span> <span class="s2">"opacity"</span><span class="p">;</span> <span class="k">to:</span> <span class="mi">1</span><span class="p">;</span> <span class="k">duration:</span> <span class="mi">450</span> <span class="p">}</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">See if you can use OpacityAnimator instead, it runs directly on the rendering thread and won't block if the application is otherwise busy</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434182#file434182line355" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/ui/main.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">355</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="nx">ScriptAction</span> <span class="p">{</span> <span class="k">script:</span> <span class="p">{</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">ScriptAction {

    script: {</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434182#file434182line375" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/ui/main.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">375</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">                    <span class="kd">var</span> <span class="nx">type</span> <span class="o">=</span> <span class="s1">'file'</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">var type = items.isDir(url) ? "folder" : "file";</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434182#file434182line402" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/ui/main.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">402</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">            <span class="k">iconSource:</span> <span class="s1">'arrow-left'</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">Prefer " instead of '</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434182#file434182line403" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/ui/main.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">403</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">            <span class="k">onClicked:</span> <span class="p">{</span> <span class="nx">nextTimer</span><span class="p">.</span><span class="nx">stop</span><span class="p">();</span> <span class="nx">previousItem</span><span class="p">()</span> <span class="p">}</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">onClicked: {

    nextTimer.stop();

    previousItem()

}</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434182#file434182line417" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/ui/main.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">417</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">            <span class="k">anchors.bottomMargin:</span> <span class="mi">6</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">Avoid using hardcoded pixel sizes, use eg. units.smallSpacing</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434182#file434182line445" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/ui/main.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">445</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="c1">// </span><span class="cs">BUG</span><span class="c1"> </span><span class="cs">TODO</span><span class="c1"> Fix overlay so _all_ mouse events reach lower components</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">I guess if you placed the MouseArea behind (ie. code-wise, above) the buttons, it would just work?</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434182#file434182line454" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/ui/main.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">454</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">            <span class="k">onEntered:</span> <span class="p">{</span></pre></td>

  </tr>


  <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">455</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">                <span class="nx">overlay</span><span class="p">.</span><span class="nx">opacity</span> <span class="o">=</span> <span class="mi">1</span></pre></td>

  </tr>


  <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">456</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">                <span class="k">if</span><span class="p">(</span><span class="nx">config</span><span class="p">.</span><span class="nx">pauseOnMouseOver</span><span class="p">)</span> <span class="nx">main</span><span class="p">.</span><span class="nx">pause</span> <span class="o">=</span> <span class="kc">true</span></pre></td>

  </tr>


  <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">457</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">            <span class="p">}</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">Replace this by declarative bindings:</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">eg. have the overlay use:

opacity: overlayMouseArea.containsMouse ? 1 : 0</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">and the Timer

running: whatever && !overlayMouseArea.containsMouse</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434182#file434182line500" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/ui/main.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">500</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="nx">ProgressBar</span> <span class="p">{</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">Given you completely re-style the ProgressBar, you might as well make your own component from Rectangles. This way you could use Animators to drive the animation and avoid expensive computations the ProgressBar might be doing.</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434182#file434182line585" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/package/contents/ui/main.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">585</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="nx">PlasmaComponents</span><span class="p">.</span><span class="nx">Button</span> <span class="p">{</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">I think there's a plasmoid.needsConfiguring property that does exactly that.</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434184#file434184line19" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/plugin/mediaframe.h</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">19</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="cp">#ifndef PICTURESLIDE_H</span></pre></td>

  </tr>


  <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">20</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="cp">#define PICTURESLIDE_H</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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 doesn't match the file's name</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434184#file434184line39" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/plugin/mediaframe.h</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">39</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="n">MediaFrame</span><span class="p">(</span><span class="n">QObject</span> <span class="o">*</span><span class="n">parent</span> <span class="o">=</span> <span class="mi">0</span><span class="p">);</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">nullptr instead of 0</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434184#file434184line40" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/plugin/mediaframe.h</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">40</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="o">~</span><span class="n">MediaFrame</span><span class="p">();</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">virtual ~MediaFrame();</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434184#file434184line86" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/plugin/mediaframe.h</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">86</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="kt">bool</span> <span class="n">m_random</span><span class="p">;</span></pre></td>

  </tr>


  <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">87</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="kt">int</span> <span class="n">m_next</span><span class="p">;</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">bool m_random = false;

int m_next = 0;</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434185#file434185line43" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/plugin/mediaframe.cpp</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">43</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="n">m_filters</span> <span class="o"><<</span> <span class="s">"*."</span><span class="o">+</span><span class="n">str</span><span class="p">;</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">Don't we have something re-usable elsewhere for this?</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434185#file434185line50" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/plugin/mediaframe.cpp</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">50</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="n">QObject</span><span class="o">::</span><span class="n">connect</span><span class="p">(</span><span class="o">&</span><span class="n">m_watcher</span><span class="p">,</span> <span class="n">SIGNAL</span><span class="p">(</span><span class="n">directoryChanged</span><span class="p">(</span><span class="n">QString</span><span class="p">)),</span> <span class="k">this</span><span class="p">,</span> <span class="n">SLOT</span><span class="p">(</span><span class="n">slotItemChanged</span><span class="p">(</span><span class="n">QString</span><span class="p">)));</span></pre></td>

  </tr>


  <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">51</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="n">QObject</span><span class="o">::</span><span class="n">connect</span><span class="p">(</span><span class="o">&</span><span class="n">m_watcher</span><span class="p">,</span> <span class="n">SIGNAL</span><span class="p">(</span><span class="n">fileChanged</span><span class="p">(</span><span class="n">QString</span><span class="p">)),</span> <span class="k">this</span><span class="p">,</span> <span class="n">SLOT</span><span class="p">(</span><span class="n">slotItemChanged</span><span class="p">(</span><span class="n">QString</span><span class="p">)));</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">Prefer new connect syntax:</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">connect(&m_watcher, &QFileSystemWatcher::directoryChanged, this, &MediaFrame::slotItemChanged);</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Also, you're in a QObject subclass, no need to speciy QObject::</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434185#file434185line54" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/plugin/mediaframe.cpp</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">54</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="n">MediaFrame</span><span class="o">::~</span><span class="n">MediaFrame</span><span class="p">()</span></pre></td>

  </tr>


  <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">55</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="p">{</span></pre></td>

  </tr>


  <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">56</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="p">}</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">MediaFrame::~MediaFrame() = default;</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434185#file434185line63" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/plugin/mediaframe.cpp</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">63</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">Whitespace</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434185#file434185line69" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/plugin/mediaframe.cpp</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">69</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="kt">void</span> <span class="n">MediaFrame</span><span class="o">::</span><span class="n">setRandom</span><span class="p">(</span><span class="k">const</span> <span class="kt">bool</span> <span class="o">&</span><span class="n">random</span><span class="p">)</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">just bool is sufficient, use const & for complex types</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434185#file434185line71" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/plugin/mediaframe.cpp</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">71</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="k">if</span> <span class="p">(</span><span class="n">random</span> <span class="o">!=</span> <span class="n">m_random</span><span class="p">)</span></pre></td>

  </tr>


  <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">72</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="p">{</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">if (m_random != random) {</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434185#file434185line80" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/plugin/mediaframe.cpp</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">80</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="k">if</span> <span class="p">(</span><span class="n">min</span> <span class="o">></span> <span class="n">max</span><span class="p">)</span></pre></td>

  </tr>


  <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">81</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="p">{</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">if (min > max) {</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434185#file434185line88" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/plugin/mediaframe.cpp</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">88</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="k">return</span> <span class="p">((</span><span class="n">qrand</span><span class="p">()</span><span class="o">%</span><span class="p">(</span><span class="n">max</span><span class="o">-</span><span class="n">min</span><span class="o">+</span><span class="mi">1</span><span class="p">))</span><span class="o">+</span><span class="n">min</span><span class="p">);</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">Add spaces:</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">return ((qrand() % (max - min + 1)) + min);</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434185#file434185line96" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/plugin/mediaframe.cpp</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">96</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="n">QString</span> <span class="n">MediaFrame</span><span class="o">::</span><span class="n">hash</span><span class="p">(</span><span class="n">QString</span> <span class="n">str</span><span class="p">)</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">const QString &</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434185#file434185line98" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/plugin/mediaframe.cpp</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">98</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="k">return</span> <span class="n">QString</span><span class="p">(</span> <span class="n">QCryptographicHash</span><span class="o">::</span><span class="n">hash</span><span class="p">(</span> <span class="n">str</span><span class="p">.</span><span class="n">toUtf8</span><span class="p">(),</span> <span class="n">QCryptographicHash</span><span class="o">::</span><span class="n">Md5</span><span class="p">).</span><span class="n">toHex</span><span class="p">()</span> <span class="p">);</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">QString::fromLatin1 or so?</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434185#file434185line108" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/plugin/mediaframe.cpp</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">108</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="k">return</span> <span class="p">(</span><span class="n">isDir</span><span class="p">(</span><span class="n">path</span><span class="p">)</span> <span class="o">&&</span> <span class="n">QDir</span><span class="p">(</span><span class="n">path</span><span class="p">).</span><span class="n">entryInfoList</span><span class="p">(</span><span class="n">QDir</span><span class="o">::</span><span class="n">NoDotAndDotDot</span><span class="o">|</span><span class="n">QDir</span><span class="o">::</span><span class="n">AllEntries</span><span class="p">).</span><span class="n">count</span><span class="p">()</span> <span class="o">==</span> <span class="mi">0</span><span class="p">);</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">Is there no faster way than to create a list of files just to check if it's empty?</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Also there's an .isEmpty() method</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434185#file434185line114" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/plugin/mediaframe.cpp</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">114</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="c1">// Check if the file exists and not a directory</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">Doesn't static QFile::exists() do that? I don't know if it returns true for folders, though.</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434185#file434185line120" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/plugin/mediaframe.cpp</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">120</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="n">add</span><span class="p">(</span><span class="n">path</span><span class="p">,</span> <span class="nb">false</span><span class="p">);</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">Avoid the boolean trap [1], I have no idea what "false" means if I don't look up the function signature to find out it means "non-recursive", use an enum instead.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">[1] http://ariya.ofilabs.com/2011/08/hall-of-api-shame-boolean-trap.html</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434185#file434185line125" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/plugin/mediaframe.cpp</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">125</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="k">if</span><span class="p">(</span><span class="n">has</span><span class="p">(</span><span class="n">path</span><span class="p">))</span></pre></td>

  </tr>


  <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">126</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="p">{</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">if (…) {</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Also, has() isn't a particularly descriptive name</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434185#file434185line131" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/plugin/mediaframe.cpp</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">131</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="n">QUrl</span> <span class="n">url</span> <span class="o">=</span> <span class="n">QUrl</span><span class="p">(</span><span class="n">path</span><span class="p">);</span></pre></td>

  </tr>


  <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">132</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="n">QString</span> <span class="n">localPath</span> <span class="o">=</span> <span class="n">url</span><span class="p">.</span><span class="n">toString</span><span class="p">(</span><span class="n">QUrl</span><span class="o">::</span><span class="n">PreferLocalFile</span><span class="p">);</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">You're creating an url just to make a local file from it again?</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434185#file434185line142" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/plugin/mediaframe.cpp</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">142</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">            <span class="n">QDirIterator</span> <span class="n">dirIterator</span><span class="p">(</span><span class="n">localPath</span><span class="p">,</span> <span class="n">m_filters</span><span class="p">,</span> <span class="n">QDir</span><span class="o">::</span><span class="n">Files</span><span class="p">,</span> <span class="p">(</span><span class="n">recursive</span> <span class="o">?</span> <span class="n">QDirIterator</span><span class="o">::</span><span class="n">Subdirectories</span> <span class="o">|</span> <span class="n">QDirIterator</span><span class="o">::</span><span class="nl">FollowSymlinks</span> <span class="p">:</span> <span class="n">QDirIterator</span><span class="o">::</span><span class="n">NoIteratorFlags</span><span class="p">));</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">I wonder if this should be done in another thread, have a look at [plasma-workspace]/wallpapers/image/image.cpp where it does that.</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434185#file434185line176" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/plugin/mediaframe.cpp</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">176</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="n">m_pathMap</span><span class="p">[</span><span class="n">path</span><span class="p">]</span> <span class="o">=</span> <span class="n">paths</span><span class="p">;</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">m_pathMap.insert(path, paths)</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434185#file434185line201" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/plugin/mediaframe.cpp</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">201</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="n">m_pathMap</span> <span class="o">=</span> <span class="n">QHash</span><span class="o"><</span><span class="n">QString</span><span class="p">,</span> <span class="n">QStringList</span><span class="o">></span><span class="p">();</span></pre></td>

  </tr>


  <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">202</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="n">m_allFiles</span> <span class="o">=</span> <span class="n">QStringList</span><span class="p">();</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">use .clear() instead</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434185#file434185line242" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/plugin/mediaframe.cpp</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">242</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="kt">void</span> <span class="n">MediaFrame</span><span class="o">::</span><span class="n">get</span><span class="p">(</span><span class="n">QJSValue</span> <span class="n">successCallback</span><span class="p">,</span> <span class="n">QJSValue</span> <span class="n">errorCallback</span><span class="p">)</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">We don't use the "callback" battern.</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434185#file434185line250" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/plugin/mediaframe.cpp</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">250</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="k">if</span><span class="p">(</span><span class="n">size</span> <span class="o"><</span> <span class="mi">1</span><span class="p">)</span></pre></td>

  </tr>


  <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">251</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="p">{</span></pre></td>

  </tr>


  <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">252</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="k">if</span><span class="p">(</span><span class="n">size</span> <span class="o">==</span> <span class="mi">0</span><span class="p">)</span></pre></td>

  </tr>


  <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">253</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">        <span class="p">{</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <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;">I guess this could be rewritten to be clearer. Also, coding style.</p></pre>

 </div>

</div>

<br />


<div>


<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/126793/diff/1/?file=434185#file434185line300" style="color: black; font-weight: bold; text-decoration: underline;">applets/mediaframe/plugin/mediaframe.cpp</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">300</font></th>

    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">            <span class="n">QString</span> <span class="n">cachedFile</span> <span class="o">=</span> <span class="n">getCacheDirectory</span><span class="p">()</span><span class="o">+</span><span class="s">"/"</span><span class="o">+</span><span class="n">hash</span><span class="p">(</span><span class="n">path</span><span class="p">)</span><span class="o">+</span><span class="s">"_"</span><span class="o">+</span><span class="n">m_filename</span><span class="p">;</span></pre></td>

  </tr>


 </tbody>


</table>


 <div style="margin-left: 2em;">


  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Use QLatin1Char('/') and QLatin1Char('_')</pre>

 </div>

</div>

<br />


<p>- Kai Uwe Broulik</p>


<br />

<p>On Januar 18th, 2016, 11:27 vorm. UTC, Lars Pontoppidan 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.</div>

<div>By Lars Pontoppidan.</div>


<p style="color: grey;"><i>Updated Jan. 18, 2016, 11:27 vorm.</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;">KF5 version of the (Picture) Frame widget</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;">kdeplasma-addons builds on Kubuntu 15.10.</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/CMakeLists.txt <span style="color: grey">(ed938f8)</span></li>


 <li>applets/mediaframe/CMakeLists.txt <span style="color: grey">(PRE-CREATION)</span></li>


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


 <li>applets/mediaframe/package/contents/code/utility.js <span style="color: grey">(PRE-CREATION)</span></li>


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


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


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


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


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


 <li>applets/mediaframe/package/metadata.desktop <span style="color: grey">(PRE-CREATION)</span></li>


 <li>applets/mediaframe/plugin/mediaframe.h <span style="color: grey">(PRE-CREATION)</span></li>


 <li>applets/mediaframe/plugin/mediaframe.cpp <span style="color: grey">(PRE-CREATION)</span></li>


 <li>applets/mediaframe/plugin/mediaframeplugin.h <span style="color: grey">(PRE-CREATION)</span></li>


 <li>applets/mediaframe/plugin/mediaframeplugin.cpp <span style="color: grey">(PRE-CREATION)</span></li>


 <li>applets/mediaframe/plugin/qmldir <span style="color: grey">(PRE-CREATION)</span></li>


</ul>


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


  </td>

 </tr>

</table>


  </div>

 </body>

</html>