Details

    • Sub-task
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 3.9.5
    • 4.0.0
    • Others
    • Hide

      This issue should be tested in an iOS device.

      1. In Moodle web as admin, go to "Site administration > Users > Privacy and policies > Policy settings" and put a valid URL in "Site policy URL". Make sure that the URL can be embedded in an iframe. For example you can use this URL (it belongs to a map).
      2. In the app, login as a user that hasn't accepted the policy in that site. Check that you're sent to a page to accept the policy.
      3. Check that the page you put in the Site policy URL is displayed in an iframe, and that iframe fills as much height as possible. Before this patch there was some unused space below the buttons.
      Show
      This issue should be tested in an iOS device. In Moodle web as admin, go to "Site administration > Users > Privacy and policies > Policy settings" and put a valid URL in "Site policy URL". Make sure that the URL can be embedded in an iframe. For example you can use this URL (it belongs to a map). In the app, login as a user that hasn't accepted the policy in that site. Check that you're sent to a page to accept the policy. Check that the page you put in the Site policy URL is displayed in an iframe, and that iframe fills as much height as possible. Before this patch there was some unused space below the buttons.
    • MOODLE_39_STABLE
    • MOODLE_400_STABLE
    • Moodle App 4.0

    Description

      When a user needs to accept the site policy, we display it in an iframe if possible. But the height of this iframe is too small.

      This can be tested in browser and iOS (in Android emulator for some reason the head request to determine the mimetype of the policy page returns an error).

      Attachments

        1. screenshot-3.png
          screenshot-3.png
          254 kB
        2. screenshot-2.png
          screenshot-2.png
          323 kB
        3. screenshot-1.png
          screenshot-1.png
          131 kB

        Activity

          People

            dpalou Dani Palou
            dpalou Dani Palou
            Pau Ferrer Pau Ferrer
            Juan Leyva Juan Leyva
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:
              22/Apr/22