[wp-trac] [WordPress Trac] #37661: A New Experience for Discovering, Installing, and Previewing Themes in the Customizer
WordPress Trac
noreply at wordpress.org
Tue Oct 4 06:37:33 UTC 2016
#37661: A New Experience for Discovering, Installing, and Previewing Themes in the
Customizer
-------------------------------------+-------------------------------------
Reporter: celloexpressions | Owner: celloexpressions
Type: feature request | Status: assigned
Priority: high | Milestone: 4.7
Component: Customize | Version: 4.2
Severity: normal | Resolution:
Keywords: has-patch has- | Focuses: ui, accessibility,
screenshots needs-testing has- | javascript
user-testing ux-feedback |
-------------------------------------+-------------------------------------
Comment (by celloexpressions):
Thanks for the feedback, [attachment:37661.6.diff] fixes the following
issues:
- Fix panel title and header
- Fix div and ul as child element of ul
- Fix duplicate ids in theme controls across sections
- Correct spacing in favorites section
- Add spacing before favorites input
- Delete button should float left on mobile
- Focus the search input when clicking on the section header (including
icon)
- Click to collapse expandable sections
- Add aria-expanded attributes for expandable sections
- Create toggles for filter sections - note, we need confirmation that the
button with aria-expanded, within the legend element, is the best approach
for this in terms of accessibility
The remaining known issues are:
- ScrollTop themes list when changing sections
- Update installed status within details modal - check all selectors in
updates.js for customize
- Mobile loading indicator in header
- Header text/button jumps on fadeIn
- Screenshots don’t render before scroll on mobile
- Themes container may not be moved after filters in markup correctly
until panel is closed and reopened
We need a final decision on:
- Remove install without previewing buttons (except in the details modal)?
- Straight to themes on mobile, or filters first?
- Revisit the check mark - note that it's needed to distinguish "selected"
from the hover/focus state
> 'Browse all WordPress.org themes' makes me think perhaps better wording
would be 'Browse all themes on WordPress.org'? I note this isn't on the
wp-admin themes section and seems natural without it.
If we say "on WordPress.org", it might cause some confusion since you're
actually browsing them from on your site/in WP. We can't compare to the
admin screen here since installed and available are merged into one
interface, but I like this approach to distinguishing them personally.
> It seems a little odd that you can click the 'installed'. It's only one
and clicking doesn't do anything.
Not sure what you mean. Installed is its own section like the other
filters, so clicking on it opens that section (unless it's already open,
as with the others).
--
Ticket URL: <https://core.trac.wordpress.org/ticket/37661#comment:69>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list