WordPress.org

Make WordPress Core

Opened 8 years ago

Closed 8 years ago

#25866 closed defect (bug) (fixed)

Ajaxurl should be defined in the customizer

Reported by: celloexpressions Owned by: SergeyBiryukov
Milestone: 3.8 Priority: normal
Severity: normal Version:
Component: Customize Keywords: has-patch
Focuses: Cc:

Description

Our documentation says that ajaxurl is defined everywhere in wp-admin. It is not in the customizer, and we need it for the suggest script for Twenty Fourteen's featured content tag name field. Very silly for themes to do this manually.

Attachments (1)

25866.diff (2.0 KB) - added by celloexpressions 8 years ago.
Define ajaxurl directly in the customizer <head>, as it is in the rest of the admin. Also remove the code that was adding it in Twenty Fourteen.

Download all attachments as: .zip

Change History (8)

#1 @SergeyBiryukov
8 years ago

  • Milestone changed from Awaiting Review to 3.8

@celloexpressions
8 years ago

Define ajaxurl directly in the customizer <head>, as it is in the rest of the admin. Also remove the code that was adding it in Twenty Fourteen.

#2 @celloexpressions
8 years ago

  • Keywords has-patch added; needs-patch removed

#3 @lancewillett
8 years ago

  • Cc lancewillett added

#4 @rachelbaker
8 years ago

  • Cc rachel@… added

#6 @SergeyBiryukov
8 years ago

In 26082:

Define ajaxurl in the customizer.

props celloexpressions.
see #25866.

#7 @SergeyBiryukov
8 years ago

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

In 26083:

Remove the code for setting ajaxurl from Twenty Fourteen.

props celloexpressions.
fixes #25866.

Note: See TracTickets for help on using tickets.