Make WordPress Core

Opened 6 years ago

Closed 6 years ago

#12237 closed defect (bug) (fixed)


Reported by: nacin Owned by:
Milestone: 3.0 Priority: normal
Severity: normal Version:
Component: Administration Keywords: dev-feedback has-patch
Focuses: Cc:


Defining STYLE_DEBUG by itself does not serve concatenated development scripts, because both scripts and styles depend on the same concatenation global (and overriding CONCATENATE_SCRIPTS constant).

It will serve the development version of colors, but that's about it.

I suggest we get rid of STYLE_DEBUG and change all of its references to SCRIPT_DEBUG.

The alternative would be to allow concatenation to be controlled independently, but that would introduce another global and change existing behavior, having an effect outside of debugging. Removing STYLE_DEBUG has no real downside and makes it one less constant for developers to define.

Attachments (1)

12237.diff (1.7 KB) - added by nacin 6 years ago.

Download all attachments as: .zip

Change History (5)

6 years ago

#1 @nacin
6 years ago

  • Keywords has-patch added
  • Milestone changed from Unassigned to 3.0

Likewise, because of the $concatenated_scripts bottleneck, setting only SCRIPT_DEBUG will unconcatenate styles, but leave them minified.

#2 @nacin
6 years ago

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

(In [13183]) Remove STYLE_DEBUG constant, SCRIPT_DEBUG now handles both scripts and CSS. Fixes #12237

#3 @matveb
6 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Shouldn't wp-admin/css/ms.dev.css be included when SCRIPT_DEBUG?

#4 @nacin
6 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed
Note: See TracTickets for help on using tickets.