Skip to content

RTL display fixes and Hebrew translation fixes#27

Merged
mgage merged 2 commits into
mgage:multilingualfrom
taniwallach:tani_multilingual_minor_RTL_hebrew_fixes2
May 23, 2019
Merged

RTL display fixes and Hebrew translation fixes#27
mgage merged 2 commits into
mgage:multilingualfrom
taniwallach:tani_multilingual_minor_RTL_hebrew_fixes2

Conversation

@taniwallach

Copy link
Copy Markdown
  1. Fix some display issues when using an overall RTL direction of the HTML pages.
  • Student Progress:
    • Problem score/wrong attempts within <pre> preformated blocks does not behave in RTL pages.
    • Wrap in <span> with dir="rtl" fixes the display of the score/attempts data per student.
    • The table header line with the sequence of problem numbers is still displayed backwards, but I as not successful in fixing that.
  • Past answers:
    • Past answers get mangled (in particular the math) when displayed in RTL pages.
    • Have the entire table of answers set to use dir="rtl" which fixes the problem.
  • These changes should not have any effect on pages whose overall direction is LTR.
  1. Updates to heb.po - should not bother anyone not using Hebrew, and fixes typos/bugs found since the initial translation work was done.

like Hebrew or Arabic.
The data from
	lib/WeBWorK/ContentGenerator/Instructor/ShowAnswers.pm
   (using table formatting)
and	lib/WeBWorK/ContentGenerator/Instructor/StudentProgress.pm
   which are preformatted using a <pre> tag
are not displated properly when the overall HTML document in in RTL direction.
Force this data to be displayed LTR using an HTML dir attribute.
For StudentProgress data an enveloping SPAN tag was needed.
no changes added to commit (use "git add" and/or "git commit -a")
[tani@webwork webwork2]$ git commit lib/WeBWorK/ContentGenerator/Instructor/ShowAnswers.pm lib/WeBWorK/ContentGenerator/Instructor/StudentProgress.pm
@mgage

mgage commented May 23, 2019

Copy link
Copy Markdown
Owner

Looks ok. Thanks Tani.

@mgage mgage merged commit 34793ec into mgage:multilingual May 23, 2019
@taniwallach taniwallach deleted the tani_multilingual_minor_RTL_hebrew_fixes2 branch May 24, 2019 10:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants