pkp / classic

An official theme for OJS 3.1.1+
GNU General Public License v2.0
11 stars 27 forks source link

Missing support for R-to-L translation #96

Open israelcefrin opened 2 years ago

israelcefrin commented 2 years ago

Describe the bug When a Journal uses a locale for languages that are R-to-L read, e.g. ar-IQ, the theme doesn't change the text direction.

To Reproduce Steps to reproduce the behavior:

  1. Enable ar-IQ or any R-to-L locale
  2. Go to the public interface
  3. Click on the locale that was installed and enabled
  4. See error

Expected behavior It is expected that all text content changes the reading direction.

Screenshots

Desktop (please complete the following information):

Smartphone (please complete the following information):

Additional context I've added a workaround in this journal: https://jmlph.net/index.php/jmlph, including an IF/ELSE to inject dir="rtl" class="rtl" in the HTML tag. However, it is required to check every R-to-L language and, maybe, create an array for verification.