-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
View options: Distraction free is the only option to trigger a snackbar notice #66364
Comments
Hi @afercia 👋
Let me know your thoughts on this. Thanks |
To add some context, the snackbar was added for distraction free mode because it can be triggered in various conditions besides this toggle. We should definitely keep only one from the action. So, if we want consistency @Vrishabhsk 's approach to add to the others is the way only way to go. |
I don't think a notice is useful when triggered from the menu. Maybe the command palette should be the one triggering these. |
I'm not sure why the snackbar notice should sometimes appear and sometimes not for similar actions. I'm not sure that's any helpful from an user perspective. it may seem smart to contributors because we, as developers and designers, we know how these features work under the hood. However, to me, it doesn't make sense from an user perspective and it's just confusing. Consistency is key. To me, these actions should either always trigger a notice or never. |
This started with distraction free mode because it can be toggled based on other triggers and the user should be notified about the 'automated' change.
If all similar menu items don't trigger a notice can be considered consistent too and it makes sense. |
Yeah, that's gonna be fixed in the linked PR. |
Description
In the top bar 'Options' ellipsis menu, the first 4 options are related to 'view modes' of the editor:
The 'Distraction free' mode is the only one to trigger a snackbar notice. The other ones don't.
The snackbar notice is a little unexpected to me. Also, not sure how the 'Undo' control in the notice is any valuable, given the Options menu is still open and users can just click again there to undo.
On top of that, activating and deactivating these options triggers an audible
speak
message for screen reader users. However, the message for distraction free mode is duplicated. Actually, two different components trigger a message for this same feature. Also, the wording of the message is inconsistent.Screenshot when dumping all the
speak
messages to the console:Proposal
Either all these options should trigger a snackbar notice or none.
I'd like to propose to remove the snackbar notice of the Distraction free mode for consistency and simplicity. I'm not sure this snackbar notice adds much value.
Note: recommended to check what happens when toggling these options via the Command palette.
Step-by-step reproduction instructions
Screenshots, screen recording, code snippet
No response
Environment info
No response
Please confirm that you have searched existing issues in the repo.
Please confirm that you have tested with all plugins deactivated except Gutenberg.
Please confirm which theme type you used for testing.
The text was updated successfully, but these errors were encountered: