Make WordPress Core

Opened 16 years ago

Closed 16 years ago

#12733 closed enhancement (fixed)

Merge Twenty Ten print style into style.css

Reported by: demetris Owned by: iammattthomas
Priority: normal Milestone: 3.0
Component: Themes Version: 3.0
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

The attached patch merges 2010’s print.css into the main CSS file.

It does this to save an extra HTTP request.

This is considered good practice for speedy websites by speed maniacs (like your humble servant) and also by authoritative sources like: http://developer.yahoo.com/performance/rules.html

Putting the print rules into the main CSS is less compatible, but to a negligible degree: As far as I know, there is some problem with IE for Mac. The major usual suspect, IE for Windows, supports this technique since version 5.

NOTE:

I noticed that, in long web pages in Firefox, Twenty Ten only prints part of the document. This is independent of the merge and will have to be looked at separately.

Attachments (1)

t12733-twentyten-print-style-merge.patch (4.5 KB ) - added by demetris 16 years ago.
Merge Twenty Ten print style into style.css

Download all attachments as: .zip

Change History (5)

@demetris
16 years ago

Merge Twenty Ten print style into style.css

#1 @nacin
16 years ago

  • Owner set to iammattthomas
  • Status newassigned

+1. Was looking at this the other day and never got around to ticketing it.

#2 @iammattthomas
16 years ago

  • Resolutionfixed
  • Status assignedclosed

(In [13872]) Merge twentyten print style into style.css. Fixes #12733. Props demetris.

#3 @jamescollins
16 years ago

  • Resolution fixed
  • Status closedreopened

trunk/wp-content/themes/twentyten/print.css should probably be deleted from SVN, rather than just being emptied.

#4 @iammattthomas
16 years ago

  • Resolutionfixed
  • Status reopenedclosed

Thanks, nice catch. fixed in r13873.

Note: See TracTickets for help on using tickets.