The default print.css does not set a suitable font-size, neither does it remove links not needed when printing.
The result is that when printing with firefox 2.0.0.3 (Ubuntu Feisty) the text is miniscule, being on the borderline of
unreadable. Also, the presence of links like "edit" and "click here to add comments" aren't pretty.
Thus, totally clueless on CSS, I hacked up this patch which does the following:
- Sets the font family to a serif font (much more readable on paper).
- Sets the font size to 11 pt (printer drivers do all weird things with pixel-based sizes, points are more predictable).
- Hides meta/links/tabs/breadcrumbs.
- Fixes a typo (sidebar-left instead of the obviously wrong sideber-left).
This makes pages nice and readable when printed, IMHO. An example would be http://edmundsson.se/node/68 (a recipe in Swedish) (you obviously have to "Print Preview" the page).
Comments on the CSS stuff:
- I suck at CSS, so there might be much more correct/elegant ways to do this.
- I would really like the logo image/text to be much smaller, say half the size, but I failed miserably doing this.
Anyhow, I would suggest that something that has the same effect as this is added to the default print.css in order to make the print results actually readable and nice :-)
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | printcss.patch | 815 bytes | cburschka |
| printcss.diff | 592 bytes | znikke |
Comments
Comment #1
cburschkaWhen attaching a patch, you'll get faster reviews if you mark the status of the issue accordingly. :)
I'd "+1" this but this practice has been denounced lately in favor of more productive follow up.
So:
- patch applies with offset of 2 to current head, works as advertised.
- hiding useless edit links I agree with, the breadcrumb I'm not so sure about. It's not just about links, but also about an idea where you are on the site. Retaining some information about the sitemap may be good even in print.
- font size and serif is good.
- typo fixing ditto
---
I notice that your patch was not made according to http://drupal.org/patch/create because I couldn't apply it with -p0 from the Drupal root without having to explicitly enter the filename themes/garland/print.css. I've rerolled your (unmodified) change for the current head of 6.x.
Comment #2
znikke commentedI didn't consider this a patch, more like "my first random whacking at CSS" and as such I suspected that it needed serious hacking before committing.
Also, I had totally missed the page about patch-creation, sorry for that. I'm still trying to get my head around this thing, and submitting issues I find as I stumble along.
Breadcrumbs or not is probably a matter of taste. The perfect solution would probably involve a preference setting on the site/page/whatever that selects what gets included on the print version, but that's a rather large project I assume ;)
For now, commented examples on what to change to include/exclude them in print.css is probably a good thing.
Anyway, I hope it eventually gets commited in some form or another.
Comment #3
catchComment #4
Stefan Nagtegaal commentedThere is a fix for this at http://drupal.org/node/231115. Please take some time to review and test it, and report back to me there.
Once http://drupal.org/node/231115 is committed, I'll close this issue..
Stefan
Comment #5
Stefan Nagtegaal commentedFixed by http://drupal.org/node/231115
Comment #6
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.