WordPress.org
Get WordPress

Make WordPress Core

Opened 14 years ago

Closed 14 years ago

#22052 closed defect (bug) (duplicate)

Add `no-customize-support` to default body classes

Reported by: drewapicture's profile DrewAPicture Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.4
Component: Toolbar Keywords: has-patch
Focuses: Cc:

Description

In order for hide-if-no-customize in the Toolbar to work, no-customize-support has to be added by default to the body classes. If there is Customize support, wp_customize_support_script() rewrites it to customize-support.

Related: #20751

Patch attached.

Attachments (2)

22052.diff​ (438 bytes) - added by DrewAPicture 14 years ago.
22052.2.diff​ (535 bytes) - added by DrewAPicture 14 years ago.
braces

Download all attachments as: .zip

Change History (6)

@DrewAPicture
14 years ago

#1 follow-up: @SergeyBiryukov
14 years ago

The formatting in 22052.diff​ is confusing. Should no-customize-support only be added if is_admin_bar_showing()? In that case, the braces are missing.

@DrewAPicture
14 years ago

braces

#2 in reply to: ↑ 1 @DrewAPicture
14 years ago

  • Cc xoodrew@… added

Replying to SergeyBiryukov:

Should no-customize-support only be added if is_admin_bar_showing()? In that case, the braces are missing.

The only instance of hide-if-no-customize on the front-end is on the Customize link in the Toolbar, so if it isn't showing, the class is moot. 22052.2.diff​ adds the braces.

#3 @SergeyBiryukov
14 years ago

  • Milestone changed from Awaiting Review to 3.5

#4 @obenland
14 years ago

  • Milestone 3.5 deleted
  • Resolution set to duplicate
  • Status changed from new to closed

Duplicate of #20896.

Note: See TracTickets for help on using tickets.