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

Ensure that existing third party themes still work with navigation 4.0 changes

XMLWordPrintable

    • MOODLE_400_STABLE
    • MOODLE_400_STABLE
    • MDL-73393-master
    • Hide

      Requires elevated privileges

      Setup:

      1. A course with an activity in it
      2. Install and the Fordson theme following the instructions https://docs.moodle.org/311/en/Installing_a_new_theme
        1. Fordson - https://moodle.org/plugins/theme_fordson
        2. In the "New settings - Fordson" page, set "Block Display Location Options" to "Single Column Boost Default"
      3. Apply the attached patch - fordson.patch (fixes a few issue with the current version of the theme)

      Testing

      1. Log in as admin
      2. Confirm you do not see an error message indicating a missing 'nav-drawer' file
      3. Go to the dashboard if not on it.
      4. Confirm you can see the hamburger icon on the top left
      5. Click on the icon
      6. Confirm you see the side bar with some content (we don't care about the content)
      7. Go to the course
      8. Go to the activity
      9. Turn on editing if not on already.
      10. Confirm you see the 'Add a block' button in the side bar.
      Show
      Requires elevated privileges Setup: A course with an activity in it Install and the Fordson theme following the instructions https://docs.moodle.org/311/en/Installing_a_new_theme Fordson - https://moodle.org/plugins/theme_fordson In the "New settings - Fordson" page, set "Block Display Location Options" to "Single Column Boost Default" Apply the attached patch - fordson.patch (fixes a few issue with the current version of the theme) Testing Log in as admin Confirm you do not see an error message indicating a missing 'nav-drawer' file Go to the dashboard if not on it. Confirm you can see the hamburger icon on the top left Click on the icon Confirm you see the side bar with some content (we don't care about the content) Go to the course Go to the activity Turn on editing if not on already. Confirm you see the 'Add a block' button in the side bar.
    • 1
    • Navigation push 14, QA time

      I've been testing the new changes with existing popular themes to ensure that we haven't broken them.

      Themes checked so far:

      • Adaptable
      • Moove
      • Fordson
      • Academi
      • Eguru
      • Snap
      • Klass

      Fordson, Academi and others are using the nav-drawer.mustache template which we have perhaps over zealously removed. I propose that we put that back. None of our code uses it, and so putting that file back will not return the nav drawer in new boost.

      It also looks like themes that use the site administration page in Boost can only see the first tab. We may want to see if we can fix that as well.

      Removing the course settings cog has left many themes with no access to course settings.

      The Snap theme looks pretty broken and I have no idea what the cause is.

      I've had another review of these third party plugins and here is the list of things to do:

      1. Put nav-drawer.mustache back.
        1. Add a deprecation comment on the file and create an issue to remove the template.
      2. Add "add a block" back to the flat navigtaion in navigationlib.php (This lets themes add navigation blocks back if needed).
        1. Deprecate the flat navigation class.

       

        1. fordson.patch
          2 kB
        2. MDL-73393_1.png
          MDL-73393_1.png
          120 kB
        3. MDL-73393_2.png
          MDL-73393_2.png
          100 kB
        4. MDL-73393_3.png
          MDL-73393_3.png
          318 kB

            peterdias Peter Dias
            abgreeve Adrian Greeve
            Mihail Geshoski Mihail Geshoski
            Victor Déniz Falcón Victor Déniz Falcón
            Angelia Dela Cruz Angelia Dela Cruz
            Votes:
            3 Vote for this issue
            Watchers:
            14 Start watching this issue

              Created:
              Updated:
              Resolved:

                Estimated:
                Original Estimate - 0 minutes
                0m
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 1 day, 1 hour, 15 minutes
                1d 1h 15m

                  Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.