[Owncloud] a patch for updating bookmarks list
golnaz.nilieh at gmail.com
Fri Aug 19 11:08:20 UTC 2011
I have applied my patch to master branch after Jan first reply,
But I will fix it now, Thanks for your guidance :)
On Fri, Aug 19, 2011 at 3:16 PM, Jan-Christoph Borchardt <
JanCBorchardt at fsfe.org> wrote:
> Oh and another thing:
> you don’t have to escape the HTML double quotes.
> On Fri, August 19, 2011 12:24, blizzz at arthur-schiwon.de wrote:
> > Hi Golnaz,
> > thanks a lot for your patch! That thing annoyed me, too, but have not yet
> > found time to fix it. So that's great!
> > Please follow Jans adivce on how to get it in :)
> > Additionally, I have had a short look into the patch and have two wishes
> > for improvement:
> > 1) could you please link the tags to ?tag=TAG ? Because they can be
> > filtered :)
> > 2) could you please use encodeEntities() when putting out foreign text to
> > the page to prevent JS insertion?
> > You can see how updateBookmarksList() utilizes both.
> > Thx + Cheers
> > Arthur
> > On Fri, 19 Aug 2011 13:55:18 +0430, Golnaz Nilieh
> > <golnaz.nilieh at gmail.com>
> > wrote:
> >> Hi,
> >> I have created a patch for bookmarks section. It shows the bookmark
> > entry
> >> in
> >> the list immediately after insertion, And hides it after deletion; using
> >> I do have write access to KDE repos, Is it ok to send a patch for each
> > fix,
> >> or I should do fixes in a separate branch and request for merging with
> >> master? In fact, It is good to have contribution policies documented.
> > _______________________________________________
> > Owncloud mailing list
> > Owncloud at kde.org
> > https://mail.kde.org/mailman/listinfo/owncloud
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the Owncloud