[wp-trac] [WordPress Trac] #47656: Remove 'Error:' prefixes from error notices as they're redundant
WordPress Trac
noreply at wordpress.org
Fri May 29 18:10:22 UTC 2020
#47656: Remove 'Error:' prefixes from error notices as they're redundant
-------------------------+-------------------------------------------------
Reporter: garrett- | Owner: audrasjb
eclipse |
Type: enhancement | Status: assigned
Priority: normal | Milestone: 5.5
Component: I18N | Version:
Severity: normal | Resolution:
Keywords: has-patch | Focuses: ui, accessibility, administration,
needs-testing | ui-copy
-------------------------+-------------------------------------------------
Changes (by sabernhardt):
* keywords: => has-patch needs-testing
Comment:
Thanks for the patch, @quadthemes!
I like most of your wording improvements, but I edited a few:
1. I corrected the verb agreement for the "Passwords don't match" error.
2. The already-registered username and email address errors now
consistently have a period before "Please" and they still use the word
"This". (If changing "This" to "That" is preferred, I have no strong
opinion either way.)
3. If we add the word "text" to the empty comment error, I think it would
be better to use the word "your": `Please type your comment text.`
The messages from `wp_get_theme()->errors()->get_error_message()`
**might** benefit from keeping the 'Error:' prefix, yet the prefix is
removed in my patch, too. (The code is found in
[https://core.trac.wordpress.org/browser/tags/5.2.2/src/wp-
admin/themes.php?marks=231#L229 wp-admin/themes.php].)
So in 47656.1.diff, these new strings are revised beyond simply removing
the Error prefix (changes in bold):
1. **Passwords don’t match.** Please enter the same password in both
password fields.
2. **Please enter** a username or email address.
3. Please type **your comment text**.
4. Site URL **you’ve entered is** already taken.
5. **There was a problem** creating site entry.
6. This email is already **registered. Please** choose another one.
7. **You can’t reply** to a comment on a draft post.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/47656#comment:15>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list