Opened 3 years ago
Closed 3 years ago
#12237 closed defect (bug) (fixed)
Merge STYLE_DEBUG with SCRIPT_DEBUG
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.0 |
| Component: | Administration | Version: | |
| Severity: | normal | Keywords: | dev-feedback has-patch |
| Cc: |
Description
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)
Change History (5)
- Resolution set to fixed
- Status changed from new to closed
- Resolution fixed deleted
- Status changed from closed to reopened
Shouldn't wp-admin/css/ms.dev.css be included when SCRIPT_DEBUG?

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