-
Notifications
You must be signed in to change notification settings - Fork 98
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Right to left support . #51
Comments
Hi @karamqubsi, RTL support would be great. Are you able to issue a PR with support for this? |
Hi @NateWr , I will try to do that , thank you :) . |
Hi @NateWr I did a small change that worked for me you might test it here (not sure if it can be suitable for a pull request ) : I'm not familiar with the ojs code base so maybe there is a better way to do that and of course it might be better for some users not to use cdn library ... anyway that fixed the problem for us , now we are using ojs3 bootstrap with rtl support , Thanks a lot for your efforts in these projects . |
Hi @karamqubsi, Thanks for your pr at #52. OJS3 actually has a config option for whether or not to enable the use of a CDN. This is important for some countries that have strict privacy laws, where people don't want the request to an external server. Instead of modifying the template file, can you instead load it in via the https://github.com/NateWr/bootstrap3/blob/master/BootstrapThreeThemePlugin.inc.php#L58 If you add the following code, this should load the RTL stylesheet if the locale is RTL, and it will load from the CDN if the
|
Hi , thanks for the help , I added a new commit to my branch please have a look , I added the code you've mentioned and I added the bootstrap-rtl.min.css to its place in the styles folder . Thanks . |
This is looking great @karamqubsi, thanks! Can you just tweak the whitespace in your latest commit? The main block in Then I think we'll be ready to go! 👍 |
Thanks @NateWr , I did the indentations and I removed the comment :) . |
I've merged that PR now. Thanks for contributing! |
Hi ,
Is it possible to implement right to left support for for Arabic , Hebrew , Persian languages . the current version looks not supporting that .
The text was updated successfully, but these errors were encountered: