Uploaded image for project: 'Moodle app'
  1. Moodle app
  2. MOBILE-1907

Handle window.open inside iframe

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.1.3
    • Fix Version/s: 3.2
    • Component/s: SCORM add-on
    • Labels:
    • Testing Instructions:
      Hide

      This needs to be tested both in Android and iOS.

      1. Login in proto 3.2.
      2. Go to Moodle Features Demo > SCORM packages > SCORM Adapt and open it.
      3. Click the top right menu inside the iframe and click Resources.
      4. Check that the first 2 resources are opened in the right app (Android) or in DocumentHandler (iOS). Check that the 3rd resource (Google) is opened in browser.
      5. Now login in proto 3.0 and go to Course with SCORMs > Reported in the forum > SCORM with links for InAppBrowser and open it.
      6. Click the link named "Link to an image with _blank". Check that the image is opened with an external app (Android) or with InAppBrowser (iOS).
      Show
      This needs to be tested both in Android and iOS. Login in proto 3.2. Go to Moodle Features Demo > SCORM packages > SCORM Adapt and open it. Click the top right menu inside the iframe and click Resources . Check that the first 2 resources are opened in the right app (Android) or in DocumentHandler (iOS). Check that the 3rd resource (Google) is opened in browser. Now login in proto 3.0 and go to Course with SCORMs > Reported in the forum > SCORM with links for InAppBrowser and open it. Click the link named "Link to an image with _blank". Check that the image is opened with an external app (Android) or with InAppBrowser (iOS).
    • Affected Branches:
      MOODLE_31_STABLE
    • Fixed Branches:
      MOODLE_32_STABLE
    • Sprint:
      Moodle Mobile 3.2

      Description

      Right now we show an error if an iframe calls window.open. We should try to handle it and open the right app with the URL.

        Attachments

          Activity

            People

            Assignee:
            dpalou Dani Palou
            Reporter:
            dpalou Dani Palou
            Peer reviewer:
            Pau Ferrer
            Integrator:
            Juan Leyva
            Tester:
            Pau Ferrer
            Participants:
            Component watchers:
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              12/Dec/16