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
    • Rank:
      44451

      Description

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

        Activity

        Hide
        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
        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
        Nadav Kavalerchik added a comment -

        Done

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

        Submitting for integration.

        Show
        Tim Hunt added a comment - Submitting for integration.
        Hide
        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
        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
        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
        Nadav Kavalerchik added a comment -

        Thanks
        I will fix MDL-35772

        Show
        Nadav Kavalerchik added a comment - Thanks I will fix MDL-35772
        Hide
        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: