media for install.css missing (patch)

[F J Kaiser]

In /core-root/wp-includes/general_template.php the function wp_admin_css is missing the @media definition. See patch.

[nacin]

Moving out of 3.0 for now. The media attribute isn't required, and if omitted, it defaults to 'all'. So setting it to 'screen' might affect some other views and devices.

[hakre]

I like the idea, patch still applies clean.

If there can be made no decision whether or not to use that (no idea if the rest of WP uses that attribute) I suggest to close the ticket as wontfix.

[F J Kaiser]

Just leave it open until there is a decision based on a discussion. No need for getting rid of this just to make the list shorter.

[DrewAPicture]

Refreshed the patch on current trunk. I'd vote to keep it screen in the patch if only because we can't guarantee all. OTOH, the script-loader uses all.

[F J Kaiser]

Replying to DrewAPicture:

Refreshed the patch on current trunk.

Remind me next time to update an open tab before making a patch :P

[mdawaffe]

What is the benefit of adding media="screen"? The rest of core uses media="all" (at least by default).