Skip to content
This repository was archived by the owner on Aug 26, 2022. It is now read-only.

Conversation

@darkwing
Copy link
Contributor

@darkwing darkwing commented Jun 1, 2012

Also added rules for hiding unnecessary print content.

@lmorchard
Copy link
Contributor

This works for me, but... should this be in a MINIFY_BUNDLES group, and included in the base.html template styles?

@darkwing
Copy link
Contributor Author

darkwing commented Jun 1, 2012

Per lmorchard's comments:

  1. I tried using MINIFY_BUNDLES first, but that only injects stylesheets with the "screen,projection,tv" medias; do you see a reason why we can't add "print" to all of them as well? If that's fine, we can add this to MINIFY_BUNDLES.
  2. I didn't add this stylesheet to base.html since I don't see much print value outside the documents themselves. I suppose it wouldn't hurt, I'll have a look.

…ry print content

Moving print stylesheet to the whole of MDN;  adding to minify list
@darkwing
Copy link
Contributor Author

darkwing commented Jun 1, 2012

Implemented both of lmorchard's ideas.

lmorchard added a commit that referenced this pull request Jun 4, 2012
Putting print stylesheet into place
@lmorchard lmorchard merged commit 3ec68b3 into mdn:master Jun 4, 2012
escattone added a commit that referenced this pull request Jul 20, 2017
* PR #243 (update legacy notice wording)
* PR #244 (Updating sidebar to reflect body)
* PR #245 (Updating the Preload specname)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants