You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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
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 thanClose 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
The text was updated successfully, but these errors were encountered: