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

          Attachments

            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:
                4 Start watching this issue

                Dates

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