Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-20501

Text alignment problem in Course Activities surveys in arabic (LTR issue)

    Details

      Description

      Text alignment problem in arabic at Course->Activities-.Surveys

      The text in first and second columns should be aligned to right as shown in screenshot

      Suggestions:
      moodle/mod/survey/Index.php
      Line No:37,40,43

      Remove code:
      $table->align = array ("CENTER", "LEFT", "CENTER");
      $table->align = array ("LEFT", "LEFT");

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            tsala Helen Foster added a comment -

            Phani, thanks for your report and suggested fix. I'm assuming it's still a problem in Moodle 2.0.

            Show
            tsala Helen Foster added a comment - Phani, thanks for your report and suggested fix. I'm assuming it's still a problem in Moodle 2.0.
            Hide
            rwijaya Rossiani Wijaya added a comment -

            Thanks Phani for reporting.

            Creating patch to fix 1.9, 2.0, 2.1, and master.

            Show
            rwijaya Rossiani Wijaya added a comment - Thanks Phani for reporting. Creating patch to fix 1.9, 2.0, 2.1, and master.
            Hide
            rajeshtaneja Rajesh Taneja added a comment -

            Looks good to me Rossie
            Thanks for providing the patch.

            Show
            rajeshtaneja Rajesh Taneja added a comment - Looks good to me Rossie Thanks for providing the patch.
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Thanks guys, this has been integrated now

            Show
            samhemelryk Sam Hemelryk added a comment - Thanks guys, this has been integrated now
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Thanks guys, tested on all branches before integration.
            Cheers
            Sam

            Show
            samhemelryk Sam Hemelryk added a comment - Thanks guys, tested on all branches before integration. Cheers Sam
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Sent upstream and closing, many thanks!

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Sent upstream and closing, many thanks!

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  10/Oct/11