Make WordPress Core

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#20403 closed task (blessed) (fixed)

UX improvements for theme browsing

Reported by: koopersmith's profile koopersmith Owned by: koopersmith's profile 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 13 years ago.

Download all attachments as: .zip

Change History (22)

#1 @koopersmith
13 years ago

  • Status changed from new to accepted

Also, removing dead CSS and basic prettification.

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

#2 @koopersmith
13 years ago

In [20411]:

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

#3 follow-up: @helenyhou
13 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.

#4 in reply to: ↑ 3 @koopersmith
13 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.

#5 @koopersmith
13 years ago

In [20412]:

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

#6 @koopersmith
13 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.

#7 @koopersmith
13 years ago

In [20415]:

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

#8 @koopersmith
13 years ago

In [20426]:

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

#30 @koopersmith
13 years ago

In [20445]:

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

#31 @koopersmith
13 years ago

In [20446]:

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

#32 @koopersmith
13 years ago

In [20455]:

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

#33 @koopersmith
13 years ago

In [20458]:

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

#34 @koopersmith
13 years ago

In [20459]:

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

#35 @koopersmith
13 years ago

In [20514]:

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

#36 @nacin
13 years ago

  • Component changed from Administration to Appearance

#37 @lightningspirit
13 years ago

  • Cc lightningspirit@… added

#38 @koopersmith
13 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.

@aaroncampbell
13 years ago

#39 @aaroncampbell
13 years ago

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

Patch added

#40 @koopersmith
13 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.

#41 @koopersmith
13 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.

#42 @ocean90
13 years ago

[20730] needs RTL support, see #20626.

Note: See TracTickets for help on using tickets.