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
Add description to the save panel header when nothing is checked #57716
Conversation
Size Change: +166 B (0%) Total Size: 1.69 MB
ℹ️ View Unchanged
|
Flaky tests detected in baeea76. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/7474526220
|
'The following changes have been made to your site, templates, and content.' | ||
) | ||
: __( | ||
'No changes will be made to your site, templates, and content. Select the items you want to save.' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is a good change in general.
Would "Select the items you wish to save."
be enough or do we need a qualifying statement?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1, really good idea having some descriptive text, but I like the idea of shortening it a little if we can 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would
"Select the items you wish to save."
be enough or do we need a qualifying statement?
Certainly, I think this is sufficient. Updated by 4b1036b👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚢
LGTM
The verb "wish" is a bit softer in English, but that's a personal bias.
The copy can always be updated later if folks agree.
Thank you!
Fixes #52689
What?
This PR adds a message to the save panel header when nothing is checked.
Why?
When no entity is checked, the save button is disabled. Some users may find it difficult to understand why they are disabled without an explanation.
How?
Added description. Props to @benwormald who suggested this message in this comment.
If you have a more appropriate message, I welcome your suggestions.
Testing Instructions