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

Site Editor: Ensure consistency between "resetting" mechanism #46859

Open
annezazu opened this issue Jan 3, 2023 · 8 comments
Open

Site Editor: Ensure consistency between "resetting" mechanism #46859

annezazu opened this issue Jan 3, 2023 · 8 comments
Labels
[Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") Needs Copy Review Needs review of user-facing copy (language, phrasing) Needs Design Feedback Needs general design feedback. [Type] Enhancement A suggestion for improvement.

Comments

@annezazu
Copy link
Contributor

annezazu commented Jan 3, 2023

What problem does this address?

As part of the nineteenth call for testing for the FSE Outreach Program, the following feedback was shared:

Why is “clear customizations” for a template in a different place (in the title area) and named differently from “reset defaults” for styles?

What is your proposed solution?

Consider using consistent language across each chance to reset something. From a quick audit, I see a few places:

  • Template list uses "clear customizations"
  • Template part list uses "clear customizations"
  • Top toolbar uses "clear customizations"
  • Design tools uses "RESET" and "Reset All"
  • Styles uses "Reset to defaults"

cc @WordPress/gutenberg-design for consideration as it's not clear to me if there's an advantage to having these each be different.

@annezazu annezazu added [Type] Enhancement A suggestion for improvement. Needs Design Feedback Needs general design feedback. Needs Copy Review Needs review of user-facing copy (language, phrasing) [Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") labels Jan 3, 2023
@provenself
Copy link

provenself commented Jan 3, 2023

Might be a little early for editorial to weigh in, but I'd opt for consistent phrasing that's universally understood — in these cases, "reset to defaults" (or "reset to default") applies across the board.

Reasoning:
• "Clear" being both a verb and an adjective might introduce some ambiguity for non-native English speakers
• Simply using "reset" is a little ominous, while "reset to defaults" communicates more clearly what the starting point would be

@annezazu
Copy link
Contributor Author

annezazu commented Jan 3, 2023

Thank you for chiming in already @provenself ❤️

@provenself
Copy link

Thank you for chiming in already @provenself ❤️

Thank you for not pointing out the typo in my response (fixed now)!

@jameskoster
Copy link
Contributor

Good ticket, I agree it would be beneficial to unify the language used in these similar actions.

For me a simple "Reset" would probably work in most cases... I feel "to defaults" is pretty much implied. For situations where it might feel ominous (it'd be good to unpack that), perhaps we can add descriptions?

@richtabor
Copy link
Member

For me a simple "Reset" would probably work in most cases.

I tend to agree.

@tresorama
Copy link

tresorama commented Feb 13, 2023

Just my 2 cents.

A common language, shared in the whole UI, is a great goal, but requires an olistich revisiting of various part of the interface, and requires time.

I think that the high priority for now is to ensure that the User doesn't have fear of pressing a button.
User need to know what will happen, so a description is required IMO.

We can avoid cluttering the UI with a tooltip on hover, instead of adding a long text in the UI.

I've done some exploration on Figma these days about this feature.

https://www.figma.com/file/wPO6umj8PYlwJX5JvVlSCE/wordpress?node-id=2%3A40&t=MVQFbuW69GdK5hmh-1

https://www.figma.com/file/wPO6umj8PYlwJX5JvVlSCE/wordpress?node-id=0%3A1&t=MVQFbuW69GdK5hmh-1

@paaljoachim
Copy link
Contributor

Comparing:
"reset" = to me is not totally clear to what it resets to.
The "reset to default" = to me says I can now reset back to before I made any adjustments (default).

I took a screenshot of this screen from @tresorama Jacopo's Figma files. Which I feels works well.

Screenshot 2023-02-13 at 20 18 48

@hanneslsm
Copy link

hanneslsm commented Sep 9, 2023

Coming from #54319

For me "Clear" is removing all settings to blank. (somehow related, the gradient should not have any colors after "clearing" them: #51102) "Clear" = "Unset"
"Reset" is setting all settings back to the original state. This means that a color, background or all settings are cleared and then set to the default again. (e.g. defined in a theme.) "Reset" = "Set to default"
If this logic applies, we actually could change the "Clear customizations" in templates to "Reset":
image

Instead of just saying "Clear", it can make sense to be more specific. I think "Clear Media" for the cover block is fine, because it just does that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") Needs Copy Review Needs review of user-facing copy (language, phrasing) Needs Design Feedback Needs general design feedback. [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

No branches or pull requests

7 participants