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

Horizontal Scroll bar problem with Arabic Interface inside a Lesson

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.4, 2.2.1, 2.3
    • Fix Version/s: 2.1.5, 2.2.2
    • Component/s: Lesson
    • Labels:
      None
    • Testing Instructions:
      Hide
      1. Create a lesson in moodle.
      2. Change the interface language to Arabic.
      3. An horizontal scroll bar shouldn't appears inside the lesson

      note: adding language to the site:

      1. site admin > language > language packs
      2. select language to install
      3. setting to display language selection on home page: site admin > language > language settings > enable display language menu
      Show
      Create a lesson in moodle. Change the interface language to Arabic. An horizontal scroll bar shouldn't appears inside the lesson note: adding language to the site: site admin > language > language packs select language to install setting to display language selection on home page: site admin > language > language settings > enable display language menu
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      For an unknown reason, using with Arabic languages the lesson is unnecessarily wider than the browser window, although there is nothing beyond the left border (left since its rtl) of the document).
      And this problem appears with the standard moodle theme also.
      I tried to disable the css but with not result.

        Gliffy Diagrams

          Activity

          Hide
          marwov Marwan Chaaya added a comment -

          It seems it is related to the left side menu of the lesson. If I choose NOT to show the left menu, the scrollbar disappear!!

          Show
          marwov Marwan Chaaya added a comment - It seems it is related to the left side menu of the lesson. If I choose NOT to show the left menu, the scrollbar disappear!!
          Hide
          rwijaya Rossiani Wijaya added a comment -

          Fixed .skip CSS class in lesson to be similar to .skip class in core.css

          Posting patch.

          Show
          rwijaya Rossiani Wijaya added a comment - Fixed .skip CSS class in lesson to be similar to .skip class in core.css Posting patch.
          Hide
          abgreeve Adrian Greeve added a comment -

          The fix looks good.
          My only suggestion is possibly to update the test instructions to include how to switch to Arabic (for dumb people like me).
          Thanks Rossie.

          Show
          abgreeve Adrian Greeve added a comment - The fix looks good. My only suggestion is possibly to update the test instructions to include how to switch to Arabic (for dumb people like me). Thanks Rossie.
          Hide
          rwijaya Rossiani Wijaya added a comment -

          Thanks Adrian for reviewing.

          Update testing instruction.

          Submitting this for integration review.

          Show
          rwijaya Rossiani Wijaya added a comment - Thanks Adrian for reviewing. Update testing instruction. Submitting this for integration review.
          Hide
          marwov Marwan Chaaya added a comment -

          Hey Rossiani, Can you explain in details how to get the fix and how to use it. I can't find .skip in lesson, and I can't see a .skip in core.css also.

          Thank you for your effort

          Show
          marwov Marwan Chaaya added a comment - Hey Rossiani, Can you explain in details how to get the fix and how to use it. I can't find .skip in lesson, and I can't see a .skip in core.css also. Thank you for your effort
          Hide
          rwijaya Rossiani Wijaya added a comment -

          Hi Marwan,

          The css file is located at mod/lesson/styles.css. Find the following line:

          .path-mod-lesson .skip {position: absolute;left: -1000em;width: 20em;}

          change it to

          .path-mod-lesson .skip {position: absolute;top: -1000em;width: 20em;}

          As admin, 'Purge all caches' and it should fix the issue.

          Show
          rwijaya Rossiani Wijaya added a comment - Hi Marwan, The css file is located at mod/lesson/styles.css. Find the following line: .path-mod-lesson .skip {position: absolute;left: -1000em;width: 20em;} change it to .path-mod-lesson .skip {position: absolute;top: -1000em;width: 20em;} As admin, 'Purge all caches' and it should fix the issue.
          Hide
          marwov Marwan Chaaya added a comment -

          I tried it, it is NOT working . I still have the same problem.
          Note, in addition to the testing instructions, the lesson should : Display left menu

          Thank you

          Show
          marwov Marwan Chaaya added a comment - I tried it, it is NOT working . I still have the same problem. Note, in addition to the testing instructions, the lesson should : Display left menu Thank you
          Hide
          rwijaya Rossiani Wijaya added a comment -

          Did you purge all caches (site admin > development > purge all caches)?

          Show
          rwijaya Rossiani Wijaya added a comment - Did you purge all caches (site admin > development > purge all caches)?
          Hide
          marwov Marwan Chaaya added a comment -

          Hey Rossiani,
          I forgot to purge all caches, my bad It is working perfectly.
          Thank you very much

          Show
          marwov Marwan Chaaya added a comment - Hey Rossiani, I forgot to purge all caches, my bad It is working perfectly. Thank you very much
          Hide
          nebgor Aparup Banerjee added a comment -

          Thanks for the work , thats been integrated. This probably needs testing in some versions.

          Marwan: on which versions did you test the patch on?

          Show
          nebgor Aparup Banerjee added a comment - Thanks for the work , thats been integrated. This probably needs testing in some versions. Marwan: on which versions did you test the patch on?
          Hide
          marwov Marwan Chaaya added a comment -

          I updated to version 2.2.1
          I don't know about previous versions.

          Show
          marwov Marwan Chaaya added a comment - I updated to version 2.2.1 I don't know about previous versions.
          Hide
          andyjdavis Andrew Davis added a comment -

          Should I be getting a horizontal scrollbar? According to the testing instructions I should but Im not. It seems that the testing instructions are probably wrong.

          Show
          andyjdavis Andrew Davis added a comment - Should I be getting a horizontal scrollbar? According to the testing instructions I should but Im not. It seems that the testing instructions are probably wrong.
          Hide
          rwijaya Rossiani Wijaya added a comment -

          Hi Andrew,

          You are right. It's a mistake on the testing instruction. The horizontal scrollbar shouldn't appear on the page.

          I updated the testing instruction.

          Thank you for testing it.

          Show
          rwijaya Rossiani Wijaya added a comment - Hi Andrew, You are right. It's a mistake on the testing instruction. The horizontal scrollbar shouldn't appear on the page. I updated the testing instruction. Thank you for testing it.
          Hide
          andyjdavis Andrew Davis added a comment -

          It now works as described. Passing.

          Show
          andyjdavis Andrew Davis added a comment - It now works as described. Passing.
          Hide
          marwov Marwan Chaaya added a comment -

          Hey Andrew and Rossiani, you should add the following to the testing instructions:
          Testing Instructions:
          1-Create a lesson in moodle.
          2-Change the interface language to Arabic.
          3-An horizontal scroll bar shouldn't appears inside the lesson
          4- THE LESSON SHOULD HAVE A MENU ENABLED FROM THE SETTINGS.
          Note: Version 1.9 is also affected.

          Show
          marwov Marwan Chaaya added a comment - Hey Andrew and Rossiani, you should add the following to the testing instructions: Testing Instructions: 1-Create a lesson in moodle. 2-Change the interface language to Arabic. 3-An horizontal scroll bar shouldn't appears inside the lesson 4- THE LESSON SHOULD HAVE A MENU ENABLED FROM THE SETTINGS. Note: Version 1.9 is also affected.
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          Well,

          I wish I said it every time
          you do the things you do.
          You always lend a helping hand,
          and I'm filled with gratitude.

          You are strong and generous
          for each and everyone one of us.
          I am eternally grateful,
          I cannot say thanks enough.

          Sorry for the (un)cool bit above, lol. Closing this as fixed. Ciao

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - Well, I wish I said it every time you do the things you do. You always lend a helping hand, and I'm filled with gratitude. You are strong and generous for each and everyone one of us. I am eternally grateful, I cannot say thanks enough. Sorry for the (un)cool bit above, lol. Closing this as fixed. Ciao

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                12/Mar/12