-
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
NUX: Allow user to re-enable tips #7039
Comments
Thanks for rustling up this issue. I have tagged it for copy as we can check what we want to have said in that action. @melchoyce mentioned not just saying enable/disable - which I agree. |
'Turn on tips' and 'Turn off tips' sounds good to me. |
I hate to be difficult here, and the copy can certainly need work. But lets be very careful and super intentional if we change the verbiage in menus on toggle. Generally if we can avoid TWO sets of copy that's superior to one set. Besides, given we have a checkmark, it might be confusing:
and
☝️ Both of those seem wrong to me. Which one should have the check mark? The alternative is to use the same pattern as the editor choice, and do this:
This takes double the space, and requires its own section, and also seems bad to me. We do it for the Visual editor and Text editor because those two require different labels in order to work. If we did the following, we could have a single menu item whose label did not change, and in my opinion is easy to read. When tips are enabled:
When tips are not enabled
|
I agree with having a single action, maybe 'enable' or 'show' works. Perhaps we can get a copy review suggestion though. |
Enable or Show or Display all work, depending on what language you're using for similar kinds of settings; just keep it consistent. (Personally, I like "Show" because it's the plainest and simplest.) |
From #6631 (comment):
Let's definitely do this. Quick mockup:
cc. @karmatosed @jasmussen
The text was updated successfully, but these errors were encountered: