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

Allow IFRAME to use XHTML document in Resource add-on

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide

      Step 1 : Add a file activity name 'Test XHTML'  and add index.xhtml files

      Step 2 : Click on the file and set as main file. After that the file name will be bold.

      Step 3 :  Set Display in Appearance tab to 'Open'

      Step 4 :  Save and display.

      Step 5 : Go to the newly created resource in the app

      Expected : We should see the content of index.xhtml "Test XHTML"

      When we click the link in the content. It will not break the app.

      Show
      Step 1 : Add a file activity name 'Test XHTML'  and add index.xhtml files Step 2 : Click on the file and set as main file. After that the file name will be bold. Step 3 :  Set Display in Appearance tab to 'Open' Step 4 :  Save and display. Step 5 : Go to the newly created resource in the app Expected : We should see the content of index.xhtml "Test XHTML" When we click the link in the content. It will not break the app.
    • Affected Branches:
      MOODLE_39_STABLE
    • Fixed Branches:
      MOODLE_39_STABLE
    • Sprint:
      Moodle App 3.9.3

      Description

      Currently,Moodle app allow resource add-on to use html document.

      I suggest that we allow xhtml document (mine type is : application/xhtml+xml).

      Since we allow iframe with xhtml document.

      We should update the treatFrameLink function to allow  the click binding event for all the links in xhtml document. because tagName is not always uppercase in xhtml so we need to update add logic for lowercase tagName.

      "Because XHTML doctype returns mixed-case tagName if the document is served as Content-Type: application/xhtml+xml"

       

       

        Attachments

          Activity

            People

            Assignee:
            vuvanhieu143 Hieu Vu
            Reporter:
            vuvanhieu143 Hieu Vu
            Peer reviewer:
            Dani Palou
            Integrator:
            Juan Leyva
            Tester:
            Isabel Renedo Rouco
            Participants:
            Component watchers:
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              30/Nov/20