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 Bug
    • Status: Closed
    • Priority: Major 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:
    • Rank:
      18160

      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.

        Activity

        Hide
        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
        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
        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
        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
        Glenn Ansley added a comment -

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

        Show
        Glenn Ansley added a comment - Kudos to Jonathan Champ <jrchamp> for helping with this.
        Hide
        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
        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
        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
        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
        Tim Hunt added a comment -

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

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

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

        Show
        Glenn Ansley added a comment - Ok. Good point. I'll test those later today and get back to you. Thanks
        Hide
        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
        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
        Tim Hunt added a comment -

        Thanks Glenn. Submitting for integration now.

        Show
        Tim Hunt added a comment - Thanks Glenn. Submitting for integration now.
        Hide
        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
        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
        Sam Hemelryk added a comment -

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

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

        Tested this issue on IE7, IE8, and IE9.

        The patch works great.

        Test passed.

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

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

        Show
        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: