On Fri, Sep 25, 2009 at 7:37 AM, Paul W. Frields <stickster@xxxxxxxxx> wrote: > On Thu, Sep 24, 2009 at 10:49:20PM -0500, Sijis Aviles wrote: >> All, >> >> I was poking through infra tickets and i ran into #1670 [1] (wiki: >> "Printable version" not suitable for printing). >> I decided to see if it can be done and this is what i came up with >> these examples: >> >> Fedora Wiki print - http://sijis.fedorapeople.org/fpo-print.html >> Fedora regular page - http://sijis.fedorapeople.org/fpo-a.html >> >> [1] https://fedorahosted.org/fedora-infrastructure/ticket/1670 > [...snip...] >> Note that commonPrint.css is for the wiki printable view. If needed, >> we don't update that css file and just let print.css handle everything >> when someone prints. >> >> I've tested all this on: >> Fedora 11 - FF 3.5.3 >> Windows XP - IE6, IE7, FF 3.5.2 >> >> Can someone review this and let me know what you think? > > Nice job. The code/preformatted areas could use a bit of padding on > the left (and maybe right). > > -- > Paul W. Frields http://paul.frields.org/ > gpg fingerprint: 3DA6 A0AC 6D58 FEC4 0233 5906 ACDB C937 BD11 3717 > http://redhat.com/ - - - - http://pfrields.fedorapeople.org/ > irc.freenode.net: stickster @ #fedora-docs, #fedora-devel, #fredlug > Paul, Thanks for looking and reviewing it. I added the following to commonPrint.css to resolve the padding problem and formatting you noticed and an the footer overlapping some text. -- pre { padding: 1em; } #footer { margin: 0px; } -- Could you look at it again and tell me if it looks correctly now? Sijis -- Fedora-websites-list mailing list Fedora-websites-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/fedora-websites-list
