-
Notifications
You must be signed in to change notification settings - Fork 4.6k
Editor: Improve offline error notices #73874
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
Conversation
|
Size Change: +19 B (0%) Total Size: 2.58 MB
ℹ️ View Unchanged
|
jsnajdr
left a comment
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.
Yep, looks like an improvement 🙂 ![]()
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the Unlinked AccountsThe following contributors have not linked their GitHub and WordPress.org accounts: @slrslr. Contributors, please read how to link your accounts to ensure your work is properly credited in WordPress releases. If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
|
Flaky tests detected in 59b46d5. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/20096388081
|
|
Thanks for the quick review, Jarda 🙌 |
|
This is so minor but why do we have a "Please," here rather than just "Please" :D |
If I were you, I wouldn't assume non-native English speakers are familiar with the intricacies of the language 😅 I've always known "Please, " and "Please " are both valid, but never really knew the difference. In my native tongue (Bulgarian) the comma is actually recommended there, which is what likely tilted the scale 😉 A quick research says that adding a comma adds emphasis on the "please", which I agree we don't really want here. I appreciate your nudge; it's always good to learn something new. Will follow-up shortly to remove the extra comma. Thanks @annezazu! |
|
Follow-up to remove the comma here: #73907 |
|
N.B.: What follows is a language detour just for the fun of it.
I can add that, in all the Romance languages I know, similar recommendations exist. Not in German for this particular situation, but overall it also uses the comma more. My inferrence over the years has been that contemporary English puts a lot of weight on commas. Often, as far as I know, it's not that their presence is wrong (they provide a synctatic, semantic or rhythmic function), but they are perceived to either emphasise too much or to break the flow of the sentence too much. Whereas in Romance languages — and I don't know about Bulgarian — I like to say about commas that you don't pronounce them, but you acknolwedge them. 😄 |
What?
This PR improves the editor save errors when offline.
Closes #67141.
Why?
When offline, the error messages were a bit vague, so we've been improving them. See #67141 for discussion.
How?
Adding an extra sentence to make next steps clearer.
Testing Instructions
Testing Instructions for Keyboard
Same.
Screenshots or screencast