Comment 5 for bug 1267266

Revision history for this message
Adam Reichold (adamreichold) wrote :

Hello Razi and Aeou,

I also think that anything but alternative 1 is cluttered and does not integrate nicely with the tabs as a means of switching documents. I also would not go beyond a label and a textual comment as meta-data.

In any case, I would recommend refactoring the bookmarks support to use a proper item model as a prerequisite since the model-view approach will make is achievable to keep bookmarks menu and dock synchronized. This would also remove the design problem of the BookmarkMenu class currently providing both presentation and storage for the bookmarks.

Best regards, Adam.