GNOME Bugzilla – Bug 723548
Port to GMenuModel
Last modified: 2018-05-22 15:29:55 UTC
I've ported evince's view and action menus to GMenuModel in wip/gmenu. It also adds "open" and "about" actions to the app menu. That branch also contains a patch which shows a traditional menubar on platforms that prefer it.
Hi, May you rebase it against master?
(In reply to comment #1) > May you rebase it against master? Sure, done.
I think this patch series does too much at once. Please split adding the legacy menubars off into a separate bug. And adding the about item to the app menu has been discussed in bug 674937 and rejected; let's not sneak this in with this unrelated bug at least; in any case changing the app menu deserves its own bug, too. I've already said I oppose moving the actions from window to app, so I oppose that patch in the series. I'll probably get to doing a full review of the port-to-gmenu patch at the weekend.
It would also help if patches are attached to bugzilla, so that we can use the splinter for review.
-- GitLab Migration Automatic Message -- This bug has been migrated to GNOME's GitLab instance and has been closed from further activity. You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.gnome.org/GNOME/evince/issues/433.