Code review comment for lp:~kalikiana/midori/folderzg

Revision history for this message
André Auzi (aauzi) wrote :

After carefully reading the zeigeist documents, it appears to me that the "subject" uri we use for bookmarks is wrong as well.

We currently use the bookmarked uri while we should probably use an uri identifying the bookmark itself.

I could not find a standard way to identify bookmarks with uri, in our case it could be something like:
midori://Bookmarks/.../bookmark_name

and uri for folders could then become:
midori://Bookmarks/.../folder_name

But that's another story, the fix can go as it is.

review: Approve

« Back to merge proposal