[wp-trac] [WordPress Trac] #52343: Site Health Page: Inactive theme message is inconsistent

WordPress Trac noreply at wordpress.org
Fri Jan 22 07:34:10 UTC 2021


#52343: Site Health Page: Inactive theme message is inconsistent
------------------------------+-----------------------------
 Reporter:  subrataemfluence  |      Owner:  (none)
     Type:  defect (bug)      |     Status:  new
 Priority:  normal            |  Milestone:  Awaiting Review
Component:  Site Health       |    Version:  5.6
 Severity:  normal            |   Keywords:  2nd-opinion
  Focuses:  administration    |
------------------------------+-----------------------------
 On the Site Health page, the Theme related messages are confusing. The way
 theme name is written in different areas is not consistent as well.

 `File: /wp-admin/includes/class-wp-site-health.php`

 **Scenario 1 (More than 2 themes available including Twenty Twenty-One):**
 **Message:**
 ----
 Your site has 1 inactive theme, other than Twenty Twenty-One, the default
 WordPress theme, and Yardrive, your active theme. We recommend removing
 any unused themes to enhance your site’s security.
 ----
 From the above message, it is clear that WordPress recommends removing all
 themes that unused. If someone removes **all** except the active one, Site
 Health then says:
 **Your site does not have any default theme.**

 This is confusing. The initial message should be more self-explanatory,
 something like:

 Your site has 1 inactive theme, other than Twenty Twenty-One, the default
 WordPress theme, and Yardrive, your active theme. We recommend keeping at
 least one theme to be used as the default along with the active theme and
 remove the rest to enhance your site's security. Default themes are used
 by WordPress automatically if anything is wrong with your chosen theme.


 **Scenario 2: (2 themes without any bundled theme)**
 **Message:**
 ----
 Your site has 1 inactive theme, other than twentytwentyone, the default
 WordPress theme, and Yardrive, your active theme. We recommend removing
 any unused themes to enhance your site's security.
 ----
 Again, this is confusing. It says "other than twentytwentyone", when I do
 not have that theme installed at all!
 Also, look at how the theme name is written in different cases.
 In the first case, it says **Twenty Twenty-One**, and for the second one,
 it becomes **twentytwentyone**. They should be consistent.

 If there is no twentytwentyone, either remove it from the message or
 include the exact theme name that is installed instead of twentytwentyone.

 Twenty Twenty-One is the default theme with WordPress 5.6, which is fine,
 but at the same time, the theme name should be used in such a way that it
 does not create any confusion in users' minds. Please note, I am talking
 about WordPress users, not developers.

 If someone has any bundled theme like twenty19 or twenty17 installed other
 than twenty21 as the default theme, either it should not complain about
 twenty21 or this should be made clear so that general WordPress users know
 that twenty21 is the default theme for WordPress 5.6 and is recommended to
 be there as the default theme.

 If the above makes sense, I would be happy to go ahead and try to build a
 patch for this.

 Regards

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/52343>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list