Details

    • Testing Instructions:
      Hide

      As a user assigned to some courses you should see a link My Courses at the bottom of the Navigation block. It should not be a link to /my/ but instead simply expand to allow you to access to the courses below.

      Show
      As a user assigned to some courses you should see a link My Courses at the bottom of the Navigation block. It should not be a link to /my/ but instead simply expand to allow you to access to the courses below.
    • Affected Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE, MOODLE_25_STABLE, MOODLE_26_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      wip-MDL-39286-master

      Description

      The My Courses sub category in the Navigation block shares a URL with the My Home link (example.moodle.com/my/).

      The main reason I want to fix this is because the My Courses one is both a link and a javascript expander to reveal the list of courses. Changing it into being just and expander would bring it into line with the two expanders directly above it (Site Pages and My Profile) and the sub-categories below those heading.

      It will change it from being like it's sub headings which, if you have the settings so that they can expand, will always (except in the cases of course badges, the first topic in a course, user forum posts, user blogs) be both links and expanders.

      But it also fixes an issue where both of these links (because they share a url) get highlighted as being the current page ("active_tree_node") at the same time.

        Attachments

          Activity

            People

            Assignee:
            bawjaws David Scotson
            Reporter:
            bawjaws David Scotson
            Peer reviewer:
            Sam Hemelryk
            Integrator:
            Damyon Wiese
            Participants:
            Component watchers:
            Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Amy Groshek, David Scotson, Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
            Votes:
            3 Vote for this issue
            Watchers:
            7 Start watching this issue

              Dates

              Created:
              Updated: