Moodle
  1. Moodle
  2. MDL-32388

Chrome: Scorm open in window, window opens minimized

    Details

    • Testing Instructions:
      Hide

      Add a SCORM to your site - doesn't matter which but here's one:
      http://moodle.org/mod/data/view.php?d=50&rid=1655
      In the settings choose to open in New Window (it's an advanced option and will be hidden by default)
      For first test set 100/100 in the height/width (100 and lower values are interpreted as %)
      2nd test use values higher than 100 eg 1024/800 (interpreted as pixels)

      Load the SCORM - (with pop-up blockers disabled) - when using Chrome the window is opened in a 0x0 box so you can't see it unless you are looking at the taskbar - the patch attached fixes this.

      It would be good to test the "fixed" code on multiple browsers to make sure no regressions are caused. - I have tested the new code in IE8/Chrome/FF

      Show
      Add a SCORM to your site - doesn't matter which but here's one: http://moodle.org/mod/data/view.php?d=50&rid=1655 In the settings choose to open in New Window (it's an advanced option and will be hidden by default) For first test set 100/100 in the height/width (100 and lower values are interpreted as %) 2nd test use values higher than 100 eg 1024/800 (interpreted as pixels) Load the SCORM - (with pop-up blockers disabled) - when using Chrome the window is opened in a 0x0 box so you can't see it unless you are looking at the taskbar - the patch attached fixes this. It would be good to test the "fixed" code on multiple browsers to make sure no regressions are caused. - I have tested the new code in IE8/Chrome/FF
    • Affected Branches:
      MOODLE_19_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_19_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull Master Branch:
      master_MDL-32388
    • Rank:
      39246

      Description

      Using 2.2.2+ (Build: 20120405)

      In chrome (latest 18) when you set the SCORM to open in a new window, the new window opens minimized. Even clicking on the window in task bar doesn't maximize the window. You have to right click the chrome window in task bar and click "maximize" in the right click menu.

        Issue Links

          Activity

          Harshal Mundiwale created issue -
          Harshal Mundiwale made changes -
          Field Original Value New Value
          Description Using 2.2.2+ (Build: 20120405)

          In chrome when you set the SCORM to open in a new window, the new window opens minimized. Even clicking on the window in task bar doesn't maximize the window. You have to right click the chrome window in task bar and click "maximize" in the right click menu.
          Using 2.2.2+ (Build: 20120405)

          In chrome (latest 18) when you set the SCORM to open in a new window, the new window opens minimized. Even clicking on the window in task bar doesn't maximize the window. You have to right click the chrome window in task bar and click "maximize" in the right click menu.
          Dan Marsden made changes -
          Fix Version/s STABLE backlog [ 10463 ]
          Labels triaged
          Affects Version/s 2.2 [ 10656 ]
          Affects Version/s 2.1 [ 10370 ]
          Affects Version/s 1.9.17 [ 11555 ]
          Affects Version/s 2.2.2 [ 11552 ]
          Dan Marsden made changes -
          Dan Marsden made changes -
          Status Open [ 1 ] Waiting for peer review [ 10012 ]
          Dan Marsden made changes -
          Testing Instructions Add a SCORM to your site - doesn't matter which but here's one:
          http://moodle.org/mod/data/view.php?d=50&rid=1655
          In the settings choose to open in New Window (it's an advanced option and will be hidden by default)
          For first test set 100/100 in the height/width (100 and lower values are interpreted as %)
          2nd test use values higher than 100 eg 1024/800 (interpreted as pixels)

          Load the SCORM - (with pop-up blockers disabled) - when using Chrome the window is opened in a 0x0 box so you can't see it unless you are looking at the taskbar - the patch attached fixes this.

          It would be good to test the "fixed" code on multiple browsers to make sure no regressions are caused. - I have tested the new code in IE8/Chrome/FF
          Dan Poltawski made changes -
          Original Estimate 0 minutes [ 0 ]
          Remaining Estimate 0 minutes [ 0 ]
          Status Waiting for peer review [ 10012 ] Peer review in progress [ 10013 ]
          Peer reviewer poltawski
          Dan Poltawski made changes -
          Status Peer review in progress [ 10013 ] Development in progress [ 3 ]
          Dan Marsden made changes -
          Status Development in progress [ 3 ] Waiting for integration review [ 10010 ]
          Sam Hemelryk made changes -
          Currently in integration Yes [ 10041 ]
          Sam Hemelryk made changes -
          Status Waiting for integration review [ 10010 ] Integration review in progress [ 10004 ]
          Integrator samhemelryk
          Sam Hemelryk made changes -
          Status Integration review in progress [ 10004 ] Waiting for testing [ 10005 ]
          Fix Version/s 1.9.18 [ 12050 ]
          Fix Version/s 2.1.6 [ 12052 ]
          Fix Version/s 2.2.3 [ 12053 ]
          Fix Version/s STABLE backlog [ 10463 ]
          Chris Follin made changes -
          Labels triaged moodlerooms partner triaged
          Michael de Raadt made changes -
          Tester ankit_frenz
          Ankit Agarwal made changes -
          Status Waiting for testing [ 10005 ] Testing in progress [ 10011 ]
          Ankit Agarwal made changes -
          Status Testing in progress [ 10011 ] Tested [ 10006 ]
          Ankit Agarwal made changes -
          Link This issue testing discovered MDL-32506 [ MDL-32506 ]
          Ankit Agarwal made changes -
          Link This issue testing discovered MDL-32507 [ MDL-32507 ]
          Dan Poltawski made changes -
          Status Tested [ 10006 ] Closed [ 6 ]
          Resolution Fixed [ 1 ]
          Currently in integration Yes [ 10041 ]
          Dan Poltawski made changes -
          Integration date 19/Apr/12
          Tim Hunt made changes -
          Link This issue has been marked as being related by MDL-34590 [ MDL-34590 ]

            People

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

              Dates

              • Created:
                Updated:
                Resolved: