Code review comment for lp:~larsu/nautilus/add-menubar-patch

Revision history for this message
Lars Karlitski (larsu) wrote :

Oops, forgot to reply about that one:

> - bookmark isn't listing the actual bookmarks, is that a bug or one of those
> items that would be more work to add with the current codebase/structure?

It would indeed be a bit more work, as the bookmark list to menu item stuff has been completely removed and the remaining bookmark functionality refactored. Probably a day's worth of work to stitch it back together.

I think not having the bookmarks in the menu is worse than not having the menu at all. So I think we should either remove the menu completely or fix it properly.

« Back to merge proposal