[RFC] Move Continuous option to View Mode submenu
Needs ReviewPublic

Authored by davidhurka on May 13 2019, 5:56 PM.

Details

Reviewers
None
Group Reviewers
Okular
VDG
Summary

This moves the Continuous option from the View menu to the
View Mode submenu. This makes the View Mode menu (a KActionMenu)
more useful in the main toolbar. Additionally, “Continuous” is explained by the context “View Mode”.

The primary intention was to give this View Mode a similar usage pattern like
the Change Colors menu (D21195), if both are added to the toolbar. See my
suggestion in Bugs 407217 and 407326.

FEATURE: 407326

Screenshot before:

Screenshot now:


and in the toolbar:

Test Plan
  • Look into View menu and test entries
  • Add View Mode menu to toolbar and test entries

Diff Detail

Repository
R223 Okular
Branch
move-continuous-to-view-mode-menu (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 11803
Build 11821: arc lint + arc unit
davidhurka created this revision.May 13 2019, 5:56 PM
Restricted Application added projects: Okular, Documentation. · View Herald TranscriptMay 13 2019, 5:56 PM
Restricted Application added subscribers: kde-doc-english, okular-devel. · View Herald Transcript
davidhurka requested review of this revision.May 13 2019, 5:56 PM
davidhurka edited the summary of this revision. (Show Details)May 13 2019, 6:04 PM

Nate suggested almost exactly this patch in Bug 407326. But instead of hiding Continuous when accessed from the menubar, I moved Continuous completely to the View Mode submenu.

What is part-viewermode.rc good for / how can I test that file? It’s not used in KDevelop...

Moving Continuous into the sub-menu amounts to a regression for menubar users since it becomes less noticeable in there. However I think there's an obvious solution: add the View Mode action to the default toobar, next to the Zoom combobox. Then all of this functionality becomes much much more noticeable for everyone, which is good because it's a part of the core functionality IMO.