Created attachment 232534 [details] [review]
Move About to the app menu

Created attachment 232535 [details] [review]
Remove Contents item from the gear menu

There is already a help menu item and having an item
named Contents that isn't related to the table of contents in
a document is very confusing.

Created attachment 232536 [details] [review]
Put dual and continuous modes into view menu

Instead of having them on the toolbar directly.

Attachment 232535 [details] pushed as 1941269 - Remove Contents item from the gear menu
Attachment 232536 [details] pushed as dd9bef7 - Put dual and continuous modes into view menu

Hey William, is there a specific reason why you didn't push the first patch from comment:1 along with the other two? I just saw that evince still doesn't have a standard appmenu with Help, About, Quit, and wanted to do something about, when I saw this ticket with your patch. I think that one should also be pushed.

The evince maintainers were against it at the time. Maybe now they will reconsider :)

(In reply to comment #6)
> The evince maintainers were against it at the time. Maybe now they will
> reconsider :)

That would've been nice to see here in a patch review, that's what it's for :) Anyway, the two menus in evince 3.10 are overcluttered, should be really checked and de-cluttered, and this seems like one of the easiest decisions on how to do that.

Now About lives in the Application's menu.