Opened 14 months ago

Closed 13 months ago

Last modified 13 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 13 months ago.

Download all attachments as: .zip

Change History (22)

  • Status changed from new to accepted

Also, removing dead CSS and basic prettification.

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

In [20411]:

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

comment:3 follow-up: ↓ 4   helenyhou14 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   koopersmith14 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.

In [20412]:

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

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.

In [20415]:

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

In [20426]:

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

In [20445]:

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

In [20446]:

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

In [20455]:

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

In [20458]:

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

In [20459]:

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

In [20514]:

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

  • Component changed from Administration to Appearance
  • Cc lightningspirit@… added
  • 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.

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

Patch added

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.
  • 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.

[20730] needs RTL support, see #20626.

Note: See TracTickets for help on using tickets.