Moodle
  1. Moodle
  2. MDL-35708

Right align user's information in quiz attempt review page , when in RTL mode (theme/standard & base)

        Details

        • Type: Improvement Improvement
        • Status: Closed
        • Priority: Minor Minor
        • Resolution: Fixed
        • Affects Version/s: 2.3, Future Dev
        • Fix Version/s: 2.2.6, 2.3.3
        • Component/s: Quiz
        • Labels:
        • Testing Instructions:
          Hide
          1. Add an RTL language package (Hebrew or Arabic or Farsi...) to Moodle (Home / ► Site administration / ► Language / ► Language packs)
          2. Switch to the RTL language you have installed by navigating to Moodle's Front-page and choosing it from the Language menu (or add "&lang=he" to the end of the URL you are currently using. If inside a course, set the RTL language from the course's setting)
          3. Navigate into any course
          4. As a Techer (or Admin), Add a Quiz.
          5. Add one or two questions to the Quiz (does not matter which, "Multiple Choose" are good)
          6. Switch to "Student" (Either by "switching role", if you are the Administrator, Or by using a predefined user with "Student" role in that specific course)
          7. Take the Quiz and make sure all the above UI errors do not show up
          8. After the Quiz is finished, run through the "Quiz review" to make sure it too looks OK.
          Show
          Add an RTL language package (Hebrew or Arabic or Farsi...) to Moodle (Home / ► Site administration / ► Language / ► Language packs) Switch to the RTL language you have installed by navigating to Moodle's Front-page and choosing it from the Language menu (or add "&lang=he" to the end of the URL you are currently using. If inside a course, set the RTL language from the course's setting) Navigate into any course As a Techer (or Admin), Add a Quiz. Add one or two questions to the Quiz (does not matter which, "Multiple Choose" are good) Switch to "Student" (Either by "switching role", if you are the Administrator, Or by using a predefined user with "Student" role in that specific course) Take the Quiz and make sure all the above UI errors do not show up After the Quiz is finished, run through the "Quiz review" to make sure it too looks OK.
        • Affected Branches:
          MOODLE_23_STABLE
        • Fixed Branches:
          MOODLE_22_STABLE, MOODLE_23_STABLE
        • Pull from Repository:
        • Pull Master Branch:
          WIP-MDL-35708-master_quiz
        • Pull Master Diff URL:
        • Rank:
          44451

          Description

          Right align user's information in quiz attempt review page , when in RTL mode (theme/standard & base)

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            Mary Evans added a comment -

            @Tim
            Any chance you could review the code here.
            Thanks
            Mary

            Show
            Mary Evans added a comment - @Tim Any chance you could review the code here. Thanks Mary
            Hide
            Permalink
            Tim Hunt added a comment -

            +1 from me. Thanks for fixing this Nadav.

            Are you able to submit this for integration, or do you need one of us to do it? Will you cherry-pick to the stable branches first?

            Show
            Tim Hunt added a comment - +1 from me. Thanks for fixing this Nadav. Are you able to submit this for integration, or do you need one of us to do it? Will you cherry-pick to the stable branches first?
            Hide
            Permalink
            Nadav Kavalerchik added a comment -

            Done

            Show
            Nadav Kavalerchik added a comment - Done
            Hide
            Permalink
            Tim Hunt added a comment -

            Submitting for integration.

            Show
            Tim Hunt added a comment - Submitting for integration.
            Hide
            Permalink
            Dan Poltawski added a comment -

            Thanks Nadav, i've integrated this to 22, 23 and master.

            Show
            Dan Poltawski added a comment - Thanks Nadav, i've integrated this to 22, 23 and master.
            Hide
            Permalink
            Rossiani Wijaya added a comment - - edited

            Hi Nadav,

            The fixed works great on 2.2, 2.3 and 2.4.

            However I noticed that on 2.2, the 'quiz navigation' is displayed on the left side. (screenshots)

            I'm aware that the issue might not part of intended changes. Please let me know if you want to add a quick fix for this or create a new issue.

            Thanks
            Rosie

            Show
            Rossiani Wijaya added a comment - - edited Hi Nadav, The fixed works great on 2.2, 2.3 and 2.4. However I noticed that on 2.2, the 'quiz navigation' is displayed on the left side. (screenshots) I'm aware that the issue might not part of intended changes. Please let me know if you want to add a quick fix for this or create a new issue. Thanks Rosie
            Hide
            Permalink
            Rossiani Wijaya added a comment -

            I'm passing this issue and creating MDL-35772 to fix the 'quiz navigation' display.

            Test passed.

            Show
            Rossiani Wijaya added a comment - I'm passing this issue and creating MDL-35772 to fix the 'quiz navigation' display. Test passed.
            Hide
            Permalink
            Nadav Kavalerchik added a comment -

            Thanks
            I will fix MDL-35772

            Show
            Nadav Kavalerchik added a comment - Thanks I will fix MDL-35772
            Hide
            Permalink
            Eloy Lafuente (stronk7) added a comment -

            Closing as fixed, many thanks for your awesome collaboration.

            Show
            Eloy Lafuente (stronk7) added a comment - Closing as fixed, many thanks for your awesome collaboration.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: