WordPress.org
Get WordPress

Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#34321 closed defect (bug) (fixed)

Embed service CSS does not load in src if WP_DEBUG is not set

Reported by: peterwilsoncc Owned by: SergeyBiryukov
Priority: normal Milestone: 4.4
Component: Embeds Version:
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

Running wordpress-develop.dev from the source directory does not load the CSS in the iframe is WP_DEBUG is not set to true.

Possible solutions:

  • the condition in print_oembed_embed_styles requires a check for $develop_src
  • the WP_DEBUG default should be do a check similar to SCRIPT_DEBUG
  • new global: STYLE_DEBUG

Attachments (1)

34321.diff​ (759 bytes ) - added by MikeHansenMe 11 years ago.

Download all attachments as: .zip

Change History (6)

#1 @peterwilsoncc
11 years ago

  • Component General → Embeds

#2 @SergeyBiryukov
11 years ago

  • Keywords needs-patch added
  • Milestone Awaiting Review → 4.4

#3 @swissspidy
11 years ago

SCRIPT_DEBUG works for scripts and styles and should be used accordingly. So let's replace the WP_DEBUG check with SCRIPT_DEBUG in print_oembed_embed_styles().

@MikeHansenMe
11 years ago

#4 @MikeHansenMe
11 years ago

  • Keywords has-patch added; needs-patch removed

#5 @SergeyBiryukov
11 years ago

  • Owner set to SergeyBiryukov
  • Resolution → fixed
  • Status new → closed

In 35228:

Embeds: In print_oembed_embed_styles(), load non-minified file if SCRIPT_DEBUG is true.

This is consistent with print_oembed_embed_scripts().

Props MikeHansenMe.
Fixes #34321.

Note: See TracTickets for help on using tickets.