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

theme_boost: on small screens do not display navigation on page load even if the it's last state was open - it overlaps the contents

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.2
    • Fix Version/s: 3.2
    • Component/s: Themes
    • Labels:
    • Testing Instructions:
      Hide

      In boost.

      Open drawer, follow a link, on large screen > 768 the drawer should stay open.

      On small screen < 768 the drawer should close after navigating.

      This is not exactly the same as the breakpoint where the content slides under the drawer because JS does not account for scrollbars, but CSS includes them.

      Show
      In boost. Open drawer, follow a link, on large screen > 768 the drawer should stay open. On small screen < 768 the drawer should close after navigating. This is not exactly the same as the breakpoint where the content slides under the drawer because JS does not account for scrollbars, but CSS includes them.
    • Affected Branches:
      MOODLE_32_STABLE
    • Fixed Branches:
      MOODLE_32_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-56795-master
    • Pull Master Diff URL:

      Description

      AFAIK, in ideal world of mobile application the application should show contents and navigation on demand.

      Currently if you are using theme boost and Log in to site with mobile browser, the flat navigation is shown to you, which overlaps the content area. IMO the dashboard should be visible by default.

      Consider a case:

      1. Log in user
      2. You should see dashboard and not expect to see navigation overlapping dashboard.
      3. Expand flat navigation to find more options
      4. Navigate to some course
      5. You expect to see course and not expanded navigation overlapping course contents.

        Attachments

          Activity

            People

            Assignee:
            damyon Damyon Wiese
            Reporter:
            rajeshtaneja Rajesh Taneja
            Peer reviewer:
            Ryan Wyllie
            Integrator:
            Dan Poltawski
            Tester:
            Rajesh Taneja
            Participants:
            Component watchers:
            Bas Brands
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

              Dates

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