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: Clarify the mysterious 'Entity' 👻 messages. #47886

Closed
afercia opened this issue Feb 8, 2023 · 8 comments · Fixed by #54333
Closed

Site Editor: Clarify the mysterious 'Entity' 👻 messages. #47886

afercia opened this issue Feb 8, 2023 · 8 comments · Fixed by #54333
Assignees
Labels
[Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). Good First Issue An issue that's suitable for someone looking to contribute for the first time [Status] In Progress Tracking issues with work in progress [Type] Bug An existing feature does not function as intended

Comments

@afercia
Copy link
Contributor

afercia commented Feb 8, 2023

Description

In the Site Editor, a couple messages used for the Snackbar and for the accessible audible messages refer to an 'Entity' 👻 .
It is not known whether that is a philosophical, metaphysical entity, a supernatural entity, or any other ontological-related concept.

  • Entity renamed.
  • Entity reverted.

Jokes apart, user-facing messages should avoid developers jargon. Such messages should be simple, concise, self-explanatory.

In other places in the editor, similar messages do distinguish the 'entity' in question and craft a simpler. more meaningful message based on whether the edited object is a template or template part. Similarly, these strings should be refactored to provide cleared messages for users.

Aside:
from a visual perspective, the Snackbar with dark gray background shows up on top of the sidebar with dark gray background so that the Snackbar shape is barely distinguishable. Not sure that's great. This should go in a separate issue though.

Step-by-step reproduction instructions

  • Go to the Site editor.
  • Create a custom template.
  • Click Templates > Manage all templates in the navigation.
  • In the templates list, click the Actions button of the custom template.
  • Choose Rename, change the template name and save.
  • Observe a Snackbar appears at the bottom left of the window.
  • Observe the Snackbar text is Entity renamed..
  • The same text is used for an accessible speak message for screen reader users.
  • Make a change to an existing default template.
  • Click Templates > Manage all templates in the navigation.
  • In the templates list, click the Actions button of the customized template.
  • Click Clear customizations.
  • Observe a Snackbar appears at the bottom left of the window.
  • Observe the Snackbar text is Entity reverted..
  • The same text is used for an accessible speak message for screen reader users.

Screenshots, screen recording, code snippet

Entity renamed

Entity reverted

Environment info

No response

Please confirm that you have searched existing issues in the repo.

Yes

Please confirm that you have tested with all plugins deactivated except Gutenberg.

Yes

@afercia afercia added [Type] Bug An existing feature does not function as intended [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). Good First Issue An issue that's suitable for someone looking to contribute for the first time [Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") labels Feb 8, 2023
@Divyansh1908
Copy link

Do you want it to be changed to template renamed and template reset ?

@afercia
Copy link
Contributor Author

afercia commented Feb 9, 2023

@Divyansh1908 thanks for looking into this.

Ideally, the messages should distinguish whether it's a template or a template part. Something along these lines:

const message =
	template.type === 'wp_template'
		? __( 'Template renamed.' )
		: __( 'Template part renamed.' );

createSuccessNotice( message, {
	type: 'snackbar',
} );

Same goes for the 'reverted' messages.

Also, I just noticed there are two more 'entity' messages:

  • An error occurred while reverting the entity.
  • An error occurred while renaming the entity.

These could be simplified by just removing the words the entity.

@Divyansh1908
Copy link

Ok, Thank you for clarifying it for me. I am currently very new to open-source contributions and hence clarifications like these make it easier to understand.
I will try resolving the issue with these hints

@prakhar2407
Copy link

@afercia Assign this issue to me, I will do it

@afercia
Copy link
Contributor Author

afercia commented Feb 10, 2023

Not sure who of you two guys I should assign this issue to 🙂 Not sure I'm supposed to 'assign' anyone in the first place, as I'm just a contributor amongst other contributors. @Divyansh1908 came first. I'd suggest to be nice each other and follow the spirit of OSS, where anyone is free to propose their own contributions. If you need any guidance, you may want to go through the contributor guide.

@jordesign
Copy link
Contributor

I cam across this in my testing of 6.3 beta 4 - it's more relevant (I think) now as the messaging is shown for both Templates and Patterns. Having clarity of what has been renamed/reverted here would be nice.

@prakhar2407 did you end up making any progress on this?

@jordesign
Copy link
Contributor

Adding a video for clarity:

Screen.Recording.2023-07-14.at.3.13.45.pm.mov

@annezazu annezazu moved this from ❓ Triage to 🐛 Punted to 6.3.1 in WordPress 6.3.x Editor Tasks Jul 14, 2023
@annezazu
Copy link
Contributor

Since this doesn't relate to 6.3 exactly and we're hitting RC1, punting this to 6.3.x. It's not necessary for this release.

@ndiego ndiego moved this from 🐛 Punted to 6.3.1 to ❓ Triage in WordPress 6.3.x Editor Tasks Aug 9, 2023
@github-actions github-actions bot added the [Status] In Progress Tracking issues with work in progress label Sep 10, 2023
@github-project-automation github-project-automation bot moved this from ❓ Triage to ✅ Done in WordPress 6.3.x Editor Tasks Sep 11, 2023
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") [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). Good First Issue An issue that's suitable for someone looking to contribute for the first time [Status] In Progress Tracking issues with work in progress [Type] Bug An existing feature does not function as intended
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

6 participants