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

Section Navigation Link

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 3.7, DEV backlog
    • Fix Version/s: None
    • Component/s: Blocks, Navigation
    • Labels:
    • Affected Branches:
      MOODLE_37_STABLE

      Description

      It would be great if the section names within the navigation block/tree and breadcrumb would be anchor links. Currently they are not links at all, just section name.

      I have found a couple related tracker issues (MDL-31842 and MDL-26226) but wanted to start one specific to this task as I'm willing to work on it.

      What I've found so far is that the course format_*{} class has a get_view_url() method. These methods look for the navigation=true option and then ignore the request, returning NULL.

      Simple enough, I commented out the logic which ignores navigation requests. Well that works to add the link but then causes side effects for the Navigation Block (each topic thinks it's active and its' contents/children should are shown, even though the children are not show).

      Why ignore get_view_url() requests if they are for navigation? I'm assuming specifically for this bug. And where can I find the logic which determines which navigation node is active and such?

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              ryan.isd196 Ryan Panning
              Participants:
              Component watchers:
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Sujith Haridasan, Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Sujith Haridasan
              Votes:
              6 Vote for this issue
              Watchers:
              6 Start watching this issue

                Dates

                Created:
                Updated: