Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add setting for view mode #747

Merged
merged 1 commit into from
Aug 19, 2021
Merged

add setting for view mode #747

merged 1 commit into from
Aug 19, 2021

Conversation

korelstar
Copy link
Member

Adds a new setting "Display mode for notes" with options "Open in edit mode" and "Open in preview mode". If you open a note, it will be shown either in edit mode or in preview mode - depending on this setting. In both cases, you can still change the mode by using the action "Preview" resp. "Edit" in the three-dots-menu in the upper-right corner.

Screenshot

Fixes #352.

@korelstar korelstar added the feature: editor Related to generic parts of the editor, e.g. loading/saving notes label Aug 7, 2021
@korelstar korelstar added this to the 4.2.0 milestone Aug 7, 2021
@stefan-niedermann
Copy link
Member

Nice :) Honest question: Should this be synvhronized by clients? Keeping the state in sync seems to be a valid goal, but the requirements might be different on a mobile device... 🤔

@korelstar
Copy link
Member Author

I thought about that, too. This is why I used exactly the same string labels for the setting and setting options like in your Android app.

With this PR, the new setting will be provided through the (settings) API. Hence, clients can use this setting for their GUI, too. But it's up to the client developer, how this setting should exactly be used in the client app.

@korelstar korelstar merged commit 43f00a7 into master Aug 19, 2021
@korelstar korelstar deleted the view-mode branch August 19, 2021 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature: editor Related to generic parts of the editor, e.g. loading/saving notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Default note view in preview mode
2 participants