I just checked 6.x-1.5 - thank you for adding the "Use these for links above the content" formatting to the CSS! Looks much better now.

I have only a minor observation: While the link is much better positioned now, it's still a bit too close to the text body. A tiny bit more room in between would be beneficial. Like this:

--- printlinks.css.bak	2009-04-17 14:27:52.000000000 -0700
+++ printlinks.css	2009-04-17 14:28:19.000000000 -0700
@@ -4,6 +4,7 @@
   /* Use these for links above the content */
   display: block;
   text-align: right;
+  padding-bottom: 10px;
   /* Use these for links in a corner of the content */
   /*
   position:relative;

You can check the result on my blog, just click on any post:

http://florin.myip.org/blog/

The separation between the printer-friendly link and the text body is more obvious, the whole page header feels less cramped this way.

Thanks!

Comments

jcnventura’s picture

Status: Active » Fixed

Thanks for the suggestion.

I have committed the fix, with a small change, instead of the fixed 10px, I used 0.5em. This works better with different font sizes.

João

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.