WordPress.org
Get WordPress

Make WordPress Core

Opened 14 years ago

Closed 14 years ago

#20404 closed task (blessed) (fixed)

Use theme customizer instead of thickbox theme preview

Reported by: koopersmith Owned by: koopersmith
Priority: normal Milestone: 3.4
Component: Administration Version: 3.4
Severity: normal Keywords:
Cc: Focuses:

Description

The now old-school thickbox theme previewer is still used on the theme install/update screens. We should use the new customizer instead and completely deprecate theme-preview.js.

Attachments (1)

20404.diff​ (13.5 KB ) - added by koopersmith 14 years ago.

Download all attachments as: .zip

Change History (6)

@koopersmith
14 years ago

#1 @koopersmith
14 years ago

  • Status new → accepted

The attached patch replaces all remaining instances of the thickbox theme previewer. It also separates the inclusion of the customize loader from the WP_Customize class. The loader is just a container that requires a JS file and a small amount of markup.

#2 @nacin
14 years ago

Looks good.

#3 follow-up: @dd32
14 years ago

Just a fly-by: Make sure you test the Bulk update as well as singular update, the bulk update runs through a iframe, whereas the latter doesn't.

#4 in reply to: ↑ 3 @koopersmith
14 years ago

Replying to dd32:

Just a fly-by: Make sure you test the Bulk update as well as singular update, the bulk update runs through a iframe, whereas the latter doesn't.

Yep, I tested bulk update — works for me. :)

#5 @koopersmith
14 years ago

  • Resolution → fixed
  • Status accepted → closed

In [20419]:

Replace all instances of thickbox theme preview with the theme customizer. fixes #20404.

  • Use theme customizer in theme install/update screens.
  • Separate the customize loader from the customizer. Use wp_customize_loader() to include the loader script and markup.
  • Deprecated: wp-admin/js/theme-preview.js is now no longer used by core.
Note: See TracTickets for help on using tickets.