I have applied my patch to master branch after Jan first reply,<br>But I will fix it now, Thanks for your guidance :)<br><br><div class="gmail_quote">On Fri, Aug 19, 2011 at 3:16 PM, Jan-Christoph Borchardt <span dir="ltr"><<a href="mailto:JanCBorchardt@fsfe.org">JanCBorchardt@fsfe.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Oh and another thing:<br>
Please always use single quotes in JavaScript (same goes for PHP). Then<br>
you don’t have to escape the HTML double quotes.<br>
<br>
<br>
On Fri, August 19, 2011 12:24, <a href="mailto:blizzz@arthur-schiwon.de">blizzz@arthur-schiwon.de</a> wrote:<br>
><br>
> Hi Golnaz,<br>
><br>
> thanks a lot for your patch! That thing annoyed me, too, but have not yet<br>
> found time to fix it. So that's great!<br>
><br>
> Please follow Jans adivce on how to get it in :)<br>
><br>
> Additionally, I have had a short look into the patch and have two wishes<br>
> for improvement:<br>
> 1) could you please link the tags to ?tag=TAG ? Because they can be<br>
> filtered :)<br>
> 2) could you please use encodeEntities() when putting out foreign text to<br>
> the page to prevent JS insertion?<br>
> You can see how updateBookmarksList() utilizes both.<br>
><br>
> Thx + Cheers<br>
> Arthur<br>
><br>
> On Fri, 19 Aug 2011 13:55:18 +0430, Golnaz Nilieh<br>
> <<a href="mailto:golnaz.nilieh@gmail.com">golnaz.nilieh@gmail.com</a>><br>
<div><div></div><div class="h5">> wrote:<br>
>> Hi,<br>
>> I have created a patch for bookmarks section. It shows the bookmark<br>
> entry<br>
>> in<br>
>> the list immediately after insertion, And hides it after deletion; using<br>
>> some javascript code.<br>
>><br>
>> I do have write access to KDE repos, Is it ok to send a patch for each<br>
> fix,<br>
>> or I should do fixes in a separate branch and request for merging with<br>
>> master? In fact, It is good to have contribution policies documented.<br>
</div></div><div><div></div><div class="h5">> _______________________________________________<br>
> Owncloud mailing list<br>
> <a href="mailto:Owncloud@kde.org">Owncloud@kde.org</a><br>
> <a href="https://mail.kde.org/mailman/listinfo/owncloud" target="_blank">https://mail.kde.org/mailman/listinfo/owncloud</a><br>
><br>
<br>
<br>
</div></div></blockquote></div><br>