[wp-trac] [WordPress Trac] #47222: Site Health Check: UI glitch leads user to confuse recommended improvements with critical issues

[WordPress Trac]

#47222: Site Health Check: UI glitch leads user to confuse recommended improvements
with critical issues
---------------------------+----------------------
 Reporter:  DavidAnderson  |       Owner:  Clorith
     Type:  defect (bug)   |      Status:  closed
 Priority:  normal         |   Milestone:  6.1
Component:  Site Health    |     Version:  5.2
 Severity:  normal         |  Resolution:  fixed
 Keywords:  has-patch      |     Focuses:  ui
---------------------------+----------------------
Changes (by Clorith):

 * status:  assigned => closed
 * resolution:   => fixed


Comment:

 In [changeset:"53950" 53950]:
 {{{
 #!CommitTicketReference repository="" revision="53950"
 Site Health: Don't show issue groups unless there are items in them.

 The Site Health Status screen groups issues into the categories `good`,
 `recommended`, and `critical` when displaying them to the end user.

 Initially, this screen would show `0 critical issues` while the checks
 were being performed, and then hide the group if no issues were discovered
 after all checks had completed, this not being an ideal user experience,
 as it is a better experience to show areas that have content, instead of
 hiding them after the fact.

 This change makes the groups hidden by default, and also changes the logic
 to see if a group should be displayed when an item is added to the list
 (as opposed to the previous approach which only did this check once every
 single test had completed).

 Props DavidAnderson, palmiak.
 Fixes #47222.
 }}}

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