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





<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On September 11th, 2013, 7:31 a.m. UTC, <b>Sinny Kumari</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">One issue
Suppose I searched for videos folder in search bar. Now, with search filter i can see videos folder in mediabrowser. Later when I click on video folder to broswe inside then it shows me blank. This is happening becasue still it is trying to search with name video. This should not happen, search should be called only for current browsing page where user typed it</pre>
 </blockquote>




 <p>On September 11th, 2013, 7:37 a.m. UTC, <b>Sinny Kumari</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Snapshots:
http://wstaw.org/m/2013/09/11/snapshot1.png (when searched for videos)
http://wstaw.org/m/2013/09/11/snapshot2.png (when clicked on filterd search i.e on videos)
</pre>
 </blockquote>





 <p>On July 30th, 2014, 11:45 a.m. UTC, <b>Sinny Kumari</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Status?</p></pre>
 </blockquote>








</blockquote>

<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 Harshit worked on it a few months back. I l check with him once too.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I would suggest not discarding this request as a very small step is needed to remove the bug and implement the feature correctly.</p></pre>
<br />










<p>- Akshay</p>


<br />
<p>On September 10th, 2013, 7:31 p.m. UTC, Akshay Ratan 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, Shantanu Tushar, Sinny Kumari, and Sujith Haridasan.</div>
<div>By Akshay Ratan.</div>


<p style="color: grey;"><i>Updated Sept. 10, 2013, 7:31 p.m.</i></p>









<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
plasma-mediacenter
</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;">Plasma Media Center now supports searching in local files and media. Used Regular expression to match the name.

Request to merge the branch akshay_localSearch to master of PMC.</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;">Works as intended.</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>browsingbackends/localfiles/localfilesabstractbackend.h <span style="color: grey">(84758e4)</span></li>

 <li>browsingbackends/localfiles/localfilesabstractbackend.cpp <span style="color: grey">(1144e49)</span></li>

 <li>browsingbackends/localfiles/localfilesabstractmodel.h <span style="color: grey">(43f50b7)</span></li>

 <li>browsingbackends/localfiles/localfilesabstractmodel.cpp <span style="color: grey">(ebfc602)</span></li>

 <li>mediaelements/mediabrowser/MediaBrowser.qml <span style="color: grey">(6ae777f)</span></li>

</ul>

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






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








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