Hi, <div>    Yes I think it would be perfect if you commit it for me. I dont yet have KDE Developers Right. </div><div><br></div><div>Also, my reason for commenting the line was if in future somebody wants to code some added feature regarding the playlist, the line might be of some help. Anyways, should I remove it if you say ? </div>
<div><br></div><div>Cheers,</div><div>Akshay Ratan<br><br><div class="gmail_quote">On Thu, May 16, 2013 at 10:45 PM, Shantanu Tushar <span dir="ltr"><<a href="mailto:shantanu@kde.org" target="_blank">shantanu@kde.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



 <div>
  <div style="font-family:Verdana,Arial,Helvetica,Sans-Serif"><div class="im">
   <table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border:1px #c9c399 solid">
    <tbody><tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="http://git.reviewboard.kde.org/r/110475/" target="_blank">http://git.reviewboard.kde.org/r/110475/</a>
     </td>
    </tr>
   </tbody></table>
   <br>



 </div><p>Ship it!</p>



 <pre style="white-space:pre-wrap;white-space:-moz-pre-wrap;white-space:-pre-wrap;white-space:-o-pre-wrap;word-wrap:break-word">Looks good. Should I commit this for you?</pre>
 <br>







<div>




<table width="100%" border="0" bgcolor="white">
 <thead>
  <tr>
   <th colspan="4" bgcolor="#F0F0F0" style="border-bottom:1px solid #c0c0c0;font-size:9pt;padding:4px 8px;text-align:left">
    <a href="http://git.reviewboard.kde.org/r/110475/diff/1/?file=144360#file144360line59" style="text-decoration:underline;font-weight:bold" target="_blank">mediaelements/playlist/PlaylistDelegate.qml</a>
    <span style="font-weight:normal">

     (Diff revision 1)

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



 
 

 <tbody>

  <tr>
    <th bgcolor="#e9eaa8" style="border-right:1px solid #c0c0c0" align="right"><font>58</font></th>
    <td bgcolor="#fdfebc" width="50%"><pre style="font-size:8pt;line-height:140%;margin:0">            <span>visible:</span> <span>listViewItem</span><span>.</span><span>ListView</span><span>.</span><span>isCurrentItem</span></pre>
</td>
    <th bgcolor="#e9eaa8" style="border-left:1px solid #c0c0c0;border-right:1px solid #c0c0c0" align="right"><font>59</font></th>
    <td bgcolor="#fdfebc" width="50%"><pre style="font-size:8pt;line-height:140%;margin:0">           <span><span>//</span> visible: listViewItem.ListView.isCurrentItem</span></pre></td>
  </tr>

 </tbody>

</table>

<pre style="margin-left:2em;white-space:pre-wrap;white-space:-moz-pre-wrap;white-space:-pre-wrap;white-space:-o-pre-wrap;word-wrap:break-word">don't use comments, just remove the line</pre>
</div>
<br>



<p>- Shantanu</p><div class="im">


<br>
<p>On May 16th, 2013, 5:07 p.m. UTC, Akshay Ratan wrote:</p>








</div><table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image:url('');background-repeat:repeat-x;border:1px black solid">
 <tbody><tr>
  <td><div class="im">

<div>Review request for Plasma, Marco Martin, Shantanu Tushar, and Sinny Kumari.</div>
<div>By Akshay Ratan.</div>


</div><p style="color:grey"><i>Updated May 16, 2013, 5:07 p.m.</i></p><div class="im">






<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">
 <tbody><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">Fixed bug 319626 ( <a href="https://bugs.kde.org/show_bug.cgi?id=319626" target="_blank">https://bugs.kde.org/show_bug.cgi?id=319626</a>) . Now, in the media playlist, "remove" sign is shown on every song , so the user has to just click that "sign" instead of clicking the song which earlier stopped the media.

So basically now the current media is not stopped when deleting an item from a playlist.
 </pre>
  </td>
 </tr>
</tbody></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">
 <tbody><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">Yes</pre>
  </td>
 </tr>
</tbody></table>



<div style="margin-top:1.5em">
 <b style="color:#575012;font-size:10pt;margin-top:1.5em">Bugs: </b>


 <a href="http://bugs.kde.org/show_bug.cgi?id=https://bugs.kde.org/show_bug.cgi?id=319626" target="_blank">https://bugs.kde.org/show_bug.cgi?id=319626</a>


</div>


<h1 style="color:#575012;font-size:10pt;margin-top:1.5em">Diffs </h1>
<ul style="margin-left:3em;padding-left:0">

 <li>mediaelements/playlist/PlaylistDelegate.qml <span style="color:grey">(449a0aa)</span></li>

</ul>

<p><a href="http://git.reviewboard.kde.org/r/110475/diff/" style="margin-left:3em" target="_blank">View Diff</a></p>







  </div></td>
 </tr>
</tbody></table>








  </div>
 </div>


<br>_______________________________________________<br>
Plasma-devel mailing list<br>
<a href="mailto:Plasma-devel@kde.org">Plasma-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/plasma-devel" target="_blank">https://mail.kde.org/mailman/listinfo/plasma-devel</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br>Akshay
</div>