[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