[wp-trac] [WordPress Trac] #20729: Improve Admin Colors CSS enqueuing

WordPress Trac wp-trac at lists.automattic.com
Tue May 22 17:20:17 UTC 2012


#20729: Improve Admin Colors CSS enqueuing
-----------------------------------------------+---------------------------
 Reporter:  westi                              |       Owner:
     Type:  defect (bug)                       |      Status:  new
 Priority:  normal                             |   Milestone:  Future
Component:  Appearance                         |  Release
 Severity:  normal                             |     Version:  3.1
 Keywords:  needs-patch westi-likes 3.5-early  |  Resolution:
-----------------------------------------------+---------------------------
Changes (by westi):

 * keywords:  needs-patch westi-likes, 3.5-early => needs-patch westi-likes
     3.5-early


Old description:

> The hacky way in which we enqueue the admin colors css means that you get
> different ordering of the CSS between the development version of the
> scripts and the script-loader sourced version of the scripts.
>
> This can cause CSS bugs which don't show when developing but do when
> running with the script loader - e.g. #16827
>
> It also makes it harder to implement a custom css concatenator.
>
> We should do something like -
> http://core.trac.wordpress.org/attachment/ticket/16827/colors-hacked-
> fixed.diff
>
> To make this properly enqueue the styles.
>
> We should also make this kind of call fire a {{{_doing_it_wrong()}}} as
> {{{true}}} is not a url :)
>
> {{{
> $styles->add( 'colors', true, array('wp-admin') );
>
> This code harks back from [7976]
> }}}

New description:

 The hacky way in which we enqueue the admin colors css means that you get
 different ordering of the CSS between the development version of the
 scripts and the script-loader sourced version of the scripts.

 This can cause CSS bugs which don't show when developing but do when
 running with the script loader - e.g. #16827

 It also makes it harder to implement a custom css concatenator.

 We should do something like -
 http://core.trac.wordpress.org/attachment/ticket/16827/colors-hacked-
 fixed.diff

 To make this properly enqueue the styles.

 We should also make this kind of call fire a {{{_doing_it_wrong()}}} as
 {{{true}}} is not a url :)

 {{{
 $styles->add( 'colors', true, array('wp-admin') );
 }}}

 This code harks back from [7976]

--

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/20729#comment:1>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list