WordPress.org

Make WordPress Core

Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#20403 closed task (blessed) (fixed)

UX improvements for theme browsing

Reported by: koopersmith Owned by: koopersmith
Milestone: 3.4 Priority: normal
Severity: normal Version: 3.4
Component: Customize Keywords: has-patch
Focuses: Cc:

Description

After going through the themes pages with Jane, we've found a few points that need a bit of polish. I didn't want to lump them together with #19910, so now we have this ticket.

On Install Themes:

  • Add action links: “Install now”, “Preview”.
  • Move author under theme name.
  • Figure out how to conceptually convey that users will be viewing both the preview and theme details.

On Manage Themes:

  • Make delete link red and less clickable.
  • Move version number to details.
  • Move author under theme name.

Attachments (1)

20403.diff (2.9 KB) - added by aaroncampbell 3 years ago.

Download all attachments as: .zip

Change History (22)

comment:1 @koopersmith3 years ago

  • Status changed from new to accepted

Also, removing dead CSS and basic prettification.

Last edited 3 years ago by koopersmith (previous) (diff)

comment:2 @koopersmith3 years ago

In [20411]:

Remove CSS involving unused '.theme-listing' selector. see #20403.

comment:3 follow-up: @helenyhou3 years ago

Can we possibly bump up to 300px wide in the list table? Making the screenshots 300x225. With more details hidden seems like it could help it be more of a visual browsing experience. That change was going to be rolled in with multiple screenshots but 99.9% sure that will be punted.

comment:4 in reply to: ↑ 3 @koopersmith3 years ago

Replying to helenyhou:

Can we possibly bump up to 300px wide in the list table? Making the screenshots 300x225.

Yes. That should happen regardless.

comment:5 @koopersmith3 years ago

In [20412]:

Remove CSS involving unused '#theme-information' selector. see #20403.

comment:6 @koopersmith3 years ago

In [20414]:

Theme browsing UX improvements. see #20403.

  • On manage/install themes pages, moved author under theme name.
  • On manage themes page, moved version number to details.

comment:7 @koopersmith3 years ago

In [20415]:

Fix showing/hiding theme details and move code from theme-preview.js to theme.js. see #20403.

comment:8 @koopersmith3 years ago

In [20426]:

Larger screenshots in theme list tables. see #20403, #19816.

comment:30 @koopersmith3 years ago

In [20445]:

CSS improvements for the 'Current Theme' section. see #20403.

comment:31 @koopersmith3 years ago

In [20446]:

Shrink theme screenshots for smaller screen resolutions. props helenyhou, see #20403.

comment:32 @koopersmith3 years ago

In [20455]:

Theme list table style improvements. Make 'delete' link red and separate from other actions. New action separators. see #20403.

comment:33 @koopersmith3 years ago

In [20458]:

Manage Themes: Improved separators (markup/styles) for current theme author/version. see #20403.

comment:34 @koopersmith3 years ago

In [20459]:

Manage Themes: Improved separators (markup/styles) for current theme options menu. see #20403.

comment:35 @koopersmith3 years ago

In [20514]:

RTL styles for theme browsing. props ocean90, fixes #20391, see #20403.

comment:36 @nacin3 years ago

  • Component changed from Administration to Appearance

comment:37 @lightningspirit3 years ago

  • Cc lightningspirit@… added

comment:38 @koopersmith3 years ago

  • Keywords needs-patch added
  • Owner changed from koopersmith to aaroncampbell
  • Status changed from accepted to assigned

The only remaining issue is adding install now / preview / details links to the install themes page. Handing off to aaroncampbell for a patch.

@aaroncampbell3 years ago

comment:39 @aaroncampbell3 years ago

  • Cc aaroncampbell added
  • Keywords has-patch added; needs-patch removed
  • Owner changed from aaroncampbell to koopersmith

Patch added

comment:40 @koopersmith3 years ago

In [20716]:

Fix markup in wp-admin/themes.php. see #20403.

  • Always print opening form tag.
  • Always ensure table content clears to the next line. This prevents table content from bunching next to the 'Available Themes' headline, which occurred when only one theme was active on a multisite install.

comment:41 @koopersmith3 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In [20730]:

Add 'Install now', 'Preview', and 'Details' links to theme install list items. fixes #20403.

comment:42 @ocean903 years ago

[20730] needs RTL support, see #20626.

Note: See TracTickets for help on using tickets.