Make WordPress Core

Opened 6 years ago

Closed 22 months ago

#13416 closed defect (bug) (wontfix)

media for install.css missing (patch)

Reported by: F J Kaiser Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.0
Component: Administration Keywords: has-patch
Focuses: ui Cc:


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

Attachments (2)

general_template-install-css.patch (926 bytes) - added by F J Kaiser 6 years ago.
13416.diff (952 bytes) - added by DrewAPicture 3 years ago.

Download all attachments as: .zip

Change History (12)

#1 @F J Kaiser
6 years ago

  • Cc 24-7@… added

#2 @F J Kaiser
6 years ago

  • Keywords has-patch added

#3 @nacin
6 years ago

  • Milestone changed from 3.0 to Future Release

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.

#4 @hakre
5 years ago

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.

#5 @F J Kaiser
5 years ago

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.

#6 @SergeyBiryukov
3 years ago

  • Component changed from General to UI
  • Keywords css install media removed

3 years ago

#7 follow-up: @DrewAPicture
3 years ago

  • Component changed from UI to Administration
  • Keywords ui-focus added
  • Milestone changed from Future Release to Awaiting Review

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.

#8 in reply to: ↑ 7 @F J Kaiser
3 years ago

Replying to DrewAPicture:

Refreshed the patch on current trunk.

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

#9 @mdawaffe
2 years ago

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

#10 @nacin
22 months ago

  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.