[wp-trac] [WordPress Trac] #34080: Use wp_star_rating() on Add Themes screen

WordPress Trac noreply at wordpress.org
Tue Sep 29 17:54:14 UTC 2015


#34080: Use wp_star_rating() on Add Themes screen
-------------------------------------------------+-------------------------
 Reporter:  SergeyBiryukov                       |      Owner:
     Type:  defect (bug)                         |     Status:  new
 Priority:  normal                               |  Milestone:  4.4
Component:  Themes                               |    Version:
 Severity:  normal                               |   Keywords:  has-patch
  Focuses:  ui, accessibility, rtl,              |
  administration                                 |
-------------------------------------------------+-------------------------
 Background: #26814

 Unlike Add Plugins screen, the Add Themes screen does not use
 `wp_star_rating()` and tries to recreate similar markup with JS.

 The markup, however, is a bit different, and has a few issues:

 * Does not work for RTL (see the before/after screenshots).
 * Requires a lot of custom CSS styling.
 * Uses different classes and is visually inconsistent with plugin ratings.
 * Does not appear to be accessible.

 We should add an `echo` argument to `wp_star_rating()` (like we did for
 `wp_terms_checklist()` in [33904]) and use the existing function instead.

 There is a slight visual change (the "based on %s ratings" string becomes
 redundant, because `wp_star_rating()` has a similar text as a title
 attribute and screen reader text).

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


More information about the wp-trac mailing list