[wp-trac] [WordPress Trac] #46734: Site Health: Add actions to all site status tests

WordPress Trac noreply at wordpress.org
Mon Apr 8 07:19:26 UTC 2019


#46734: Site Health: Add actions to all site status tests
-------------------------------------------------+-----------------------
 Reporter:  garrett-eclipse                      |       Owner:  (none)
     Type:  enhancement                          |      Status:  new
 Priority:  normal                               |   Milestone:  5.2
Component:  Administration                       |     Version:  trunk
 Severity:  normal                               |  Resolution:
 Keywords:  site-health has-patch needs-testing  |     Focuses:  ui, docs
-------------------------------------------------+-----------------------

Comment (by garrett-eclipse):

 Thanks @Clorith I appreciate the review/feedback.

 Replying to [comment:9 Clorith]:
 > The reason you want to use `%1$s` and similar is if there are multiple
 attributes within a translation string that are better suites in a
 different order. Here the order is fixed, the first element (the link)
 always has to go inside the `a` tag and the second one, the string, is
 always the second one. Adding ordered replacements here add a potential
 failure element if used incorrectly so as the coding standards don't state
 that they must be used I'd prefer the use of the fixed format when
 possible, just to be on the safe side.
 This makes sense for sure, might be nice to get that into the coding
 standards, within my updated patch I've set all static strings to use `%s`
 and left the translatable strings using the `%1$s` convention.

 > We should also make the URL to the support area translatable as
 suggested.
 I've updated the support URL to make it translatable and searched core to
 find all references I could and they were all translatable.

 And one final addition with the actions being fairly consistent I updated
 the 'Read more about why you should use HTTPS' link to use the convention
 making it an external link w/ the icon.

 Would love another review and testing, uploaded screens to illustrate
 changes.

 Other notes;
 - There's no filter for inactive themes hence why it doesn't have the same
 second action link that plugins has.
 - The 'One or more recommended modules are missing' section has a link to
 the 'the team handbook', should this be an action link?
 - Is there a handbook URL for 'Background updates are not working as
 expected'?
 - Which WP.org links should be translatable? Some like support make sense
 but others don't have variants so could be removed from the load on
 translators. Below is recon on this for `class-wp-site-health.php`;
 Translatable;
 https://make.wordpress.org/hosting/handbook/handbook/server-environment
 /#php-extensions
 https://wordpress.org/about/requirements/
 https://wordpress.org/support
 https://wordpress.org/support/article/why-should-i-use-https/
 Not Translatable;
 https://wordpress.org/support/article/debugging-in-wordpress/

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/46734#comment:10>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list