Moodle
  1. Moodle
  2. MDL-36812

Enabling grade_report_fixedstudents, shows the user's list on the left side of the page, in RTL mode (theme/standard)

    Details

    • Testing Instructions:
      Hide
      1. Switch language to Hebrew (or any RTL language)
      2. As Admin, Enable grade_report_fixedstudents
      3. Navigate into any course which has some users enrolled and some activities available
      4. Open the course's grader report and see that the user's list is right aligned (as seen in the second attached screen capture)
      Show
      Switch language to Hebrew (or any RTL language) As Admin, Enable grade_report_fixedstudents Navigate into any course which has some users enrolled and some activities available Open the course's grader report and see that the user's list is right aligned (as seen in the second attached screen capture)
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE, MOODLE_24_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      WIP-MDL-36812-master
    • Rank:
      46332

      Description

      Fixed user's list in course grader report should be right aligned in RTL mode (in theme/standard)

        Activity

        Hide
        Mary Evans added a comment -

        Looks OK to me Nadav so submitting this for integration.

        Show
        Mary Evans added a comment - Looks OK to me Nadav so submitting this for integration.
        Hide
        Sam Hemelryk added a comment -

        Sorry about the noise forget to set currently in integration to yes and had to restart this.

        Show
        Sam Hemelryk added a comment - Sorry about the noise forget to set currently in integration to yes and had to restart this.
        Hide
        Sam Hemelryk added a comment -

        Hi Nadav, just a quick check, this has a fixed version of 2.3, would you like me to backport it to 23 and 24?

        Show
        Sam Hemelryk added a comment - Hi Nadav, just a quick check, this has a fixed version of 2.3, would you like me to backport it to 23 and 24?
        Hide
        Nadav Kavalerchik added a comment -

        Thanks Sam, but no need. I will fix it. (better spent your precious time on other important issues )

        Show
        Nadav Kavalerchik added a comment - Thanks Sam, but no need. I will fix it. (better spent your precious time on other important issues )
        Hide
        Sam Hemelryk added a comment -

        Thanks Nadav, this has been integrated now and cherry-picked to all branches.

        Show
        Sam Hemelryk added a comment - Thanks Nadav, this has been integrated now and cherry-picked to all branches.
        Hide
        Nadav Kavalerchik added a comment -

        Bless you

        Show
        Nadav Kavalerchik added a comment - Bless you
        Hide
        Adrian Greeve added a comment -

        Tested on the 2.2, 2.3, 2.4 and master integration branches.
        My pages matched the fixed image that is attached to this issue.
        Test passed.

        Show
        Adrian Greeve added a comment - Tested on the 2.2, 2.3, 2.4 and master integration branches. My pages matched the fixed image that is attached to this issue. Test passed.
        Hide
        Eloy Lafuente (stronk7) added a comment -

        Changes are now upstream, thanks for your collaboration!

        If you are going to have any celebration next days, enjoy with your gang, if not, too!

        Ciao

        Show
        Eloy Lafuente (stronk7) added a comment - Changes are now upstream, thanks for your collaboration! If you are going to have any celebration next days, enjoy with your gang, if not, too! Ciao

          People

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

            Dates

            • Created:
              Updated:
              Resolved: