Comment 3 for bug 1661661

Revision history for this message
Ben Shum (bshum) wrote :

I have pulled all his changes into a collab branch and intend to do some review on it further.

He moved some inline CSS that was leftover in TPAC over into the stylesheet in addition to the addition of the custom RTL stylesheet files.

There is a hardcoded check for ar-AR that we might choose to replace in the future (especially if we finish bug 697926 to change the locale name to ar-JO). Mike suggested in IRC to make a new database column for the i18n languages to track whether they should be displayed LTR or RTL. This seems like the logical approach moving forward in case we add other languages in the future that are RTL.

Working collab branch here: collab/bshum/RTL-arabic

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/collab/bshum/RTL-arabic