[wp-trac] [WordPress Trac] #48716: Improve wording for PHP update warning // Part 2
WordPress Trac
noreply at wordpress.org
Fri Jan 17 20:17:38 UTC 2020
#48716: Improve wording for PHP update warning // Part 2
---------------------------------------+----------------------
Reporter: grafruessel | Owner: (none)
Type: enhancement | Status: new
Priority: normal | Milestone: 5.4
Component: Site Health | Version: 5.3
Severity: normal | Resolution:
Keywords: has-patch has-screenshots | Focuses: ui-copy
---------------------------------------+----------------------
Changes (by garrett-eclipse):
* keywords: needs-refresh => has-patch has-screenshots
Comment:
Thanks for the feedback everyone, I've refreshed the patch in
[https://core.trac.wordpress.org/attachment/ticket/48716/48716.4.diff
48716.4.diff] to accommodate everyone's input.
Current verbiage;
- Your site is running the current version of PHP (%s)
- Your site is running an older version of PHP (%s)
- Your site is running an older version of PHP (%s) which should be
updated
- Your site is running an older version of PHP (%s) which requires an
update
The two changes are;
1. Switching 'outdated' for 'older' due to @Clorith
[https://core.trac.wordpress.org/ticket/48716#comment:9 comment]
2. Switching 'and' for 'which' due to @SergeyBiryukov
[https://core.trac.wordpress.org/ticket/48716#comment:11 comment], this
should hopefully tie the 'should be updated' and 'requires an update' to
PHP instead of the site to remove confusion.
@marybaum I didn't adopt your suggestions as this string is to be used as
the accordion heading so brevity is key and I wanted to avoid having
multiple lines in a heading. For context on placement see this screenshot;
[[Image(https://core.trac.wordpress.org/raw-
attachment/ticket/48716/Screen%20Shot%202019-11-19%20at%2010.32.54%20AM.png)]]
I feel this is ready to commit but would appreciate your blessing @Clorith
as you're the maintainer.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/48716#comment:13>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list