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

IE9 browser not able to interact with some form elements within a quiz when "Full screen pop-up with some JavaScript security" Browser security has been set.

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.9.12
    • Fix Version/s: 1.9.14
    • Component/s: Quiz
    • Labels:
    • Testing Instructions:
      Hide

      To replicate the problem, you need to open up a quiz in IE9 that has text fields or dropdowns.
      Before you launch the quiz you'll need to do the following to make sure your browser is not in compatibility mode:

      1. Hit the ALT key once to show the menu at the top of IE9
      2. Click Tools --> Compatibility View Settings
      3. Uncheck the checkbox titled 'Display Intranet sites in compatibility mode'

      Launch the quiz. Text fields and dropdowns should not work when machine is not patched.
      After patch, it should work.

      Show
      To replicate the problem, you need to open up a quiz in IE9 that has text fields or dropdowns. Before you launch the quiz you'll need to do the following to make sure your browser is not in compatibility mode: 1. Hit the ALT key once to show the menu at the top of IE9 2. Click Tools --> Compatibility View Settings 3. Uncheck the checkbox titled 'Display Intranet sites in compatibility mode' Launch the quiz. Text fields and dropdowns should not work when machine is not patched. After patch, it should work.
    • Workaround:
      Hide

      Do not use the "Full screen pop-up with some JavaScript security" Browser security setting or advise IE 9 users to enable compatibility view while taking quizzes with this setting enabled - http://support.microsoft.com/kb/956197

      Show
      Do not use the "Full screen pop-up with some JavaScript security" Browser security setting or advise IE 9 users to enable compatibility view while taking quizzes with this setting enabled - http://support.microsoft.com/kb/956197
    • Affected Branches:
      MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_19_STABLE
    • Pull from Repository:

      Description

      If a quiz is set to use "Full screen pop-up with some JavaScript security" Browser security in the quiz settings, students accessing the quiz with the IE9 web browser may not be able to interact with certain form elements using a mouse. Clicking on a select menu does not open it, and clicking on text areas do not engage focus. Tabbing through elements seems to work, and select options can be changed with arrow key navigation.

      This only seems to occur when "Full screen pop-up with some JavaScript security" Browser security has been set for the quiz. When this is not set, the quiz works as expected.

      If you enable "compatibility view" in IE9 the form elements work as expected.

        Gliffy Diagrams

          Activity

          Hide
          kfortin Kathryn Fortin added a comment -

          We have clients having a similar issue referenced here: http://moodle.org/mod/forum/discuss.php?d=181224

          Students using Internet Explorer 9.0.x (I have 9.0.8112) enter into the exam, and when they get to a matching question (which has drop down menus to choose the correct answer) and the drop down menus are not working.

          In FF or Chrome it works fine. If you are using IE with a quiz has Browser security setting set up as "Full screen pop-up with some Javascript security".

          Show
          kfortin Kathryn Fortin added a comment - We have clients having a similar issue referenced here: http://moodle.org/mod/forum/discuss.php?d=181224 Students using Internet Explorer 9.0.x (I have 9.0.8112) enter into the exam, and when they get to a matching question (which has drop down menus to choose the correct answer) and the drop down menus are not working. In FF or Chrome it works fine. If you are using IE with a quiz has Browser security setting set up as "Full screen pop-up with some Javascript security".
          Hide
          blepoxp Glenn Ansley added a comment -

          The diff for this is not human readable so I placed a couple comments on the commit.
          https://github.com/ncsu-delta/moodle/commit/ef2e4cacea8719af17f26230c9aecfb820448f8f

          Show
          blepoxp Glenn Ansley added a comment - The diff for this is not human readable so I placed a couple comments on the commit. https://github.com/ncsu-delta/moodle/commit/ef2e4cacea8719af17f26230c9aecfb820448f8f
          Hide
          blepoxp Glenn Ansley added a comment -

          Kudos to Jonathan Champ <jrchamp> for helping with this.

          Show
          blepoxp Glenn Ansley added a comment - Kudos to Jonathan Champ <jrchamp> for helping with this.
          Hide
          timhunt Tim Hunt added a comment -

          Glenn / Jonathan,

          1. Do we also need similar changes in Moodle 2.0 and later? If so, can you make the patches. If not, why not?

          2. What browsers have you tested this in?

          Anyone else with an interest in this bug,

          3. Can you test it a bit, and report whether it works for you?

          Show
          timhunt Tim Hunt added a comment - Glenn / Jonathan, 1. Do we also need similar changes in Moodle 2.0 and later? If so, can you make the patches. If not, why not? 2. What browsers have you tested this in? Anyone else with an interest in this bug, 3. Can you test it a bit, and report whether it works for you?
          Hide
          blepoxp Glenn Ansley added a comment -

          Hi Tim,
          1. It is not necessary in 2.0 / 2.1. It uses YUI and the code is completely different.
          2. We tested it in IE9 &IE8 on Windows as well FF, Chrome, Safari on Mac

          Let me know if you need more info. I think it would be wise to find more testers as well.

          Show
          blepoxp Glenn Ansley added a comment - Hi Tim, 1. It is not necessary in 2.0 / 2.1. It uses YUI and the code is completely different. 2. We tested it in IE9 &IE8 on Windows as well FF, Chrome, Safari on Mac Let me know if you need more info. I think it would be wise to find more testers as well.
          Hide
          timhunt Tim Hunt added a comment -

          Those are excellent answers. My only worry is the omission of IE7 and possibly IE6 from your testing.

          Show
          timhunt Tim Hunt added a comment - Those are excellent answers. My only worry is the omission of IE7 and possibly IE6 from your testing.
          Hide
          blepoxp Glenn Ansley added a comment -

          Ok. Good point. I'll test those later today and get back to you. Thanks

          Show
          blepoxp Glenn Ansley added a comment - Ok. Good point. I'll test those later today and get back to you. Thanks
          Hide
          blepoxp Glenn Ansley added a comment -

          Ok. Confirmed it works in IE6 and IE7. I think i need to go wash my eyes out now.

          Show
          blepoxp Glenn Ansley added a comment - Ok. Confirmed it works in IE6 and IE7. I think i need to go wash my eyes out now.
          Hide
          timhunt Tim Hunt added a comment -

          Thanks Glenn. Submitting for integration now.

          Show
          timhunt Tim Hunt added a comment - Thanks Glenn. Submitting for integration now.
          Hide
          samhemelryk Sam Hemelryk added a comment -

          Thanks guys this has been integrated now.

          Just as a heads up Glenn I cherry-picked these changes so that I could amend the commit message to include the MDL issue number.

          Cheers
          Sam

          Show
          samhemelryk Sam Hemelryk added a comment - Thanks guys this has been integrated now. Just as a heads up Glenn I cherry-picked these changes so that I could amend the commit message to include the MDL issue number. Cheers Sam
          Hide
          samhemelryk Sam Hemelryk added a comment -

          I forgot to say thank you for the comments on the guthub branch! VERY helpful

          Show
          samhemelryk Sam Hemelryk added a comment - I forgot to say thank you for the comments on the guthub branch! VERY helpful
          Hide
          rwijaya Rossiani Wijaya added a comment -

          Tested this issue on IE7, IE8, and IE9.

          The patch works great.

          Test passed.

          Show
          rwijaya Rossiani Wijaya added a comment - Tested this issue on IE7, IE8, and IE9. The patch works great. Test passed.
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          git repositories have been updated with your awesome changes, thanks! Closing.

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - git repositories have been updated with your awesome changes, thanks! Closing.

            People

            • Votes:
              12 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

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