Opened 12 months ago

Closed 8 months ago

#20751 closed defect (bug) (fixed)

Customize toolbar node needs hide-if-no-customize

Reported by: nacin Owned by: koopersmith
Priority: normal Milestone: 3.4
Component: Appearance Version: 3.4
Severity: normal Keywords:
Cc: xoodrew@…

Description

Hide the Customize toolbar node if customize (or at least JS) is not supported.

Attachments (1)

20751-body-class.diff​ (438 bytes) - added by DrewAPicture 8 months ago.

Download all attachments as: .zip

Change History (5)

comment:1   koopersmith — 12 months ago

  • Owner set to koopersmith
  • Resolution set to fixed
  • Status changed from new to closed

In [20918]:

Hide 'Customize' toolbar item when customizer is not supported. fixes #20751, see #19910.

Removes script queue check for 'customize-loader' from wp_customize_support_script(), because we may want to check for customize-support on a page without the loader.

comment:2   nacin — 12 months ago

In [20934]:

Theme Customizer: As customize.php without a theme parameter defaults to the current theme, update wp_customize_url() to make $stylesheet optional and update references for the current theme.

see #19910, #20751, #20575.

DrewAPicture — 8 months ago

comment:3   DrewAPicture — 8 months ago

  • Cc xoodrew@… added
  • Keywords has-patch added
  • Resolution fixed deleted
  • Status changed from closed to reopened

The hide-if-no-customize class only works when it's paired with no-customize-support or customize-support which are supposed to reside in the body classes.

Patch attached.

comment:4   SergeyBiryukov — 8 months ago

  • Keywords has-patch removed
  • Resolution set to fixed
  • Status changed from reopened to closed

This ticket was closed on a completed milestone. Please open a new one.

Note: See TracTickets for help on using tickets.