Skip to content

Replace/Remove Close reviewer and sync #17586

Open
@BrayanDSO

Description

And tbh, idk why that is a command. Syncing when leaving the app is the most natural option, doesn't require a command, and can be done in the background. Also, the desktop version doesn't have something similar.

"But someone may want to use it". People may want to have a command to close and open the browser, or the note editor, or whatever, but obviously a command to every one of them isn't a good idea. A Sync command makes more sense than Close and sync

Originally posted by @BrayanDSO in #17585 (review)


Proposal: Remove Close reviewer and sync

Preferred alternative: Replace it with a Sync command.

@Arthur-Milchior is doing most of that in #17399, including a toolbar menu action. Adding a command after that should be easy

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions