Drupal has the infrastructure to support RTL languages, by including xxx-rtl.css file (if one exists), following each xxx.css file. The xxx-rtl.css file should onlt contain overrides to fix the display of RTL pages.

Apparently the Printer-friendly Pages does not implement this convention. I tried adding a print-rtl.css but it was not added to the printable page.

Please implement the RTL support, and then the attached print-rtl.css can be used to generate correct RTL printable pages.

Thanks,

EZ

CommentFileSizeAuthor
print-rtl.zip179 bytesE.Z
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jcnventura’s picture

jcnventura CreditAttribution: jcnventura commented
Status: Active » Postponed (maintainer needs more info)

Hi,

I don't really know how to test this, so I would like to ask you to test it yourself and tell me if it works before changing anything.

You just have to add the CSS file you provided to the print.tpl.php file, below the following line

<?php print $print["css"] ?>

you should add the line:

<link type="text/css" rel="stylesheet" media="all" href="/sites/all/modules/print/print-rtl.css" />

If this works for you, then I will add your file and I will figure out how to only include the above line when the language is RTL. If it doesn't then the solution is a lot more complex, and I will wait for a patch that is proven to work.

João

jcnventura’s picture

jcnventura CreditAttribution: jcnventura commented
Status: Postponed (maintainer needs more info) » Fixed

Hi,

I have found out how to test this (it was actually very simple). I have added the necessary code and committed to CVS. The next release will include it.

João

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

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