Comment 6 for bug 1267266

Hello Adam and Aeou,

Thanks for your comments.
I implemented it (not yet completed), available here [1] and as you suggested I chose option 1.

@Adam: For applying model approach to bookmark menu I have two idea:
1- (seems easier to me) connect aboutToShow signal of bookmark-menu to a slot and compute menu based on model.
2- (seems needs more work) re-implement BookmarkMenu in a way that it works similar to view classes in Qt code (i.e. connecting all signals of model to slots of BookmarkMenu and ...)

What's your idea?

[1] http://bazaar.launchpad.net/~srazi/qpdfview/bookmarks-dock/revision/1660

Best Regards,
Razi.