WordPress.org

Make WordPress Core

Opened 15 months ago

Closed 14 months ago

Last modified 14 months ago

#20403 closed task (blessed) (fixed)

UX improvements for theme browsing

Reported by: koopersmith Owned by: koopersmith
Priority: normal Milestone: 3.4
Component: Appearance Version: 3.4
Severity: normal Keywords: has-patch
Cc: lightningspirit@…, aaroncampbell

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 14 months ago.

Download all attachments as: .zip

Change History (22)

comment:1 koopersmith15 months ago

  • Status changed from new to accepted

Also, removing dead CSS and basic prettification.

Last edited 15 months ago by koopersmith (previous) (diff)

comment:2 koopersmith15 months ago

In [20411]:

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

comment:3 follow-up: helenyhou15 months 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 koopersmith15 months 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 koopersmith15 months ago

In [20412]:

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

comment:6 koopersmith15 months 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 koopersmith15 months ago

In [20415]:

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

comment:8 koopersmith14 months ago

In [20426]:

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

comment:30 koopersmith14 months ago

In [20445]:

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

comment:31 koopersmith14 months ago

In [20446]:

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

comment:32 koopersmith14 months ago

In [20455]:

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

comment:33 koopersmith14 months ago

In [20458]:

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

comment:34 koopersmith14 months ago

In [20459]:

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

comment:35 koopersmith14 months ago

In [20514]:

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

comment:36 nacin14 months ago

  • Component changed from Administration to Appearance

comment:37 lightningspirit14 months ago

  • Cc lightningspirit@… added

comment:38 koopersmith14 months 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.

aaroncampbell14 months ago

comment:39 aaroncampbell14 months ago

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

Patch added

comment:40 koopersmith14 months 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 koopersmith14 months 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 ocean9014 months ago

[20730] needs RTL support, see #20626.

Note: See TracTickets for help on using tickets.