Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-33703

Resizing a window or changing device orientation should adjust the chooser to still fit in the window

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3
    • Fix Version/s: 2.3
    • Component/s: Course, JavaScript
    • Labels:
    • Testing Instructions:
      Hide

      Test in as many browsers as feasible. As usual, it probably helps to have the JS console open.

      In desktop browsers:

      • Ensure that the browser window is as large as possible
      • Open a course
      • Open the module chooser
        • Confirm it's centered
        • Confirm that the page does not scroll
      • Resize the page to a size slightly smaller than the chooser is currently
        • Confirm that the chooser scales as required
        • Confirm that the page still does not scroll
      • Resize the page to a pretty tiny size - until the button disappear from the chooser
        • Confirm that the chooser scales as required
        • Confirm that the page now supports scrolling so that you can view the buttons
      • Resize the page up again
        • Confirm that the chooser scales as required
        • Confirm that the page no longer scrolls
      • Resize the page down again and refresh the page
        • Confirm that the chooser scales as required
        • Confirm that the page supports scrolling so that you can view the buttons

      In a mobile browser (I only have iOS to hand):

      • Open Safari in portrait mode
      • Open a course
      • Open the module chooser
        • Confirm it's centered
        • Note: Scrolling currently does work on mobile devices
      • Change to landscape orientation
        • Confirm that the chooser re-centers
      • Change back to portrait orientation
        • Confirm that the chooser re-centers
      Show
      Test in as many browsers as feasible. As usual, it probably helps to have the JS console open. In desktop browsers: Ensure that the browser window is as large as possible Open a course Open the module chooser Confirm it's centered Confirm that the page does not scroll Resize the page to a size slightly smaller than the chooser is currently Confirm that the chooser scales as required Confirm that the page still does not scroll Resize the page to a pretty tiny size - until the button disappear from the chooser Confirm that the chooser scales as required Confirm that the page now supports scrolling so that you can view the buttons Resize the page up again Confirm that the chooser scales as required Confirm that the page no longer scrolls Resize the page down again and refresh the page Confirm that the chooser scales as required Confirm that the page supports scrolling so that you can view the buttons In a mobile browser (I only have iOS to hand): Open Safari in portrait mode Open a course Open the module chooser Confirm it's centered Note: Scrolling currently does work on mobile devices Change to landscape orientation Confirm that the chooser re-centers Change back to portrait orientation Confirm that the chooser re-centers
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-33703-master-3

      Description

      This is primarily for iOS devices, but also has a use when a user resizes their browser.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              dobedobedoh Andrew Nicols added a comment -

              Just realised that this isn't actually blocked by MDL-33649 and have removed the blocker link, and updated the patch so it's not based on the commit for MDL-33649.

              Show
              dobedobedoh Andrew Nicols added a comment - Just realised that this isn't actually blocked by MDL-33649 and have removed the blocker link, and updated the patch so it's not based on the commit for MDL-33649 .
              Hide
              ankit_frenz Ankit Agarwal added a comment -

              This is broken in ff.
              If you open the activity chooser and than close it.
              It wont allow you to do any scrolling on the current page at all.

              Thanks

              Show
              ankit_frenz Ankit Agarwal added a comment - This is broken in ff. If you open the activity chooser and than close it. It wont allow you to do any scrolling on the current page at all. Thanks
              Hide
              dobedobedoh Andrew Nicols added a comment -

              That particular breakages comes from MDL-33649 rather than MDL-33703.

              Show
              dobedobedoh Andrew Nicols added a comment - That particular breakages comes from MDL-33649 rather than MDL-33703 .
              Hide
              dobedobedoh Andrew Nicols added a comment -

              The breakage described is actually due to another issue - putting this back up for PR

              Show
              dobedobedoh Andrew Nicols added a comment - The breakage described is actually due to another issue - putting this back up for PR
              Hide
              ankit_frenz Ankit Agarwal added a comment - - edited

              Hi Andrew,
              This looks good to me.
              Feel free to submit for integration.

              PS:- Horizontal resizing of the window are taken care in MDL-33728
              Thanks

              Show
              ankit_frenz Ankit Agarwal added a comment - - edited Hi Andrew, This looks good to me. Feel free to submit for integration. PS:- Horizontal resizing of the window are taken care in MDL-33728 Thanks
              Hide
              poltawski Dan Poltawski added a comment -

              Must we browser sniff for this?

              Show
              poltawski Dan Poltawski added a comment - Must we browser sniff for this?
              Hide
              dobedobedoh Andrew Nicols added a comment -

              You're right - we shouldn't really test the browser type for this one otherwise we'll be breaking things for non-mobile devices supporting orientation changes in the future.

              Have resubmitted without the test after testing on:

              • iOS Safari (iPad 5.1)
              • iOS Safari (iPhone 5.1)
              • Chrome
              • Opera
              • Firefox
              • IE9
              Show
              dobedobedoh Andrew Nicols added a comment - You're right - we shouldn't really test the browser type for this one otherwise we'll be breaking things for non-mobile devices supporting orientation changes in the future. Have resubmitted without the test after testing on: iOS Safari (iPad 5.1) iOS Safari (iPhone 5.1) Chrome Opera Firefox IE9
              Hide
              nebgor Aparup Banerjee added a comment -

              thanks, thats integrated right into the center of master.

              Show
              nebgor Aparup Banerjee added a comment - thanks, thats integrated right into the center of master.
              Hide
              rajeshtaneja Rajesh Taneja added a comment -

              Thanks for fixing this Andrew
              Works well on IE, FF, Chrome, opera, Safari and Android mobile browser and Dolphin

              NOTE:
              Screen resizing (pinch zoom in/out), doesn't allow user to move activity chooser, page slides, but activity chooser is not selectable/moving.

              Show
              rajeshtaneja Rajesh Taneja added a comment - Thanks for fixing this Andrew Works well on IE, FF, Chrome, opera, Safari and Android mobile browser and Dolphin NOTE: Screen resizing (pinch zoom in/out), doesn't allow user to move activity chooser, page slides, but activity chooser is not selectable/moving.
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              And this has been incorporated to all the weekly builds and also, to Moodle 2.3 Release Candidate 1, yay!

              Many, many thanks for your hard work!

              Ciao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - And this has been incorporated to all the weekly builds and also, to Moodle 2.3 Release Candidate 1, yay! Many, many thanks for your hard work! Ciao

                People

                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    25/Jun/12