Moodle
  1. Moodle
  2. MDL-24742

Output renderer and course format code ouput <span id="maincontent">. This results in an invalid document.

        Details

        • Type: Bug Bug
        • Status: Closed
        • Priority: Major Major
        • Resolution: Fixed
        • Affects Version/s: 2.0
        • Fix Version/s: 2.0.2
        • Component/s: Course, Libraries
        • Labels:
          None
        • URL:
          Main course page in any course using either weeks or topics format. Run the W3C validator - or similar.
        • Affected Branches:
          MOODLE_20_STABLE
        • Fixed Branches:
          MOODLE_20_STABLE

          Description

          Last line of the header( ) function in outputrenderers.php generates a span with id "maincontent". This is apparently used as as target for the "Skip Navigation" link (useful for screen-reader and mobile users).

          In the course directory, the "format.php" file in both the weeks and topics formats also generate a span with id = "maincontent". The existence of two tags with the same id in a document causes a validation error.

            Gliffy Diagrams

              Issue Links

              has a non-specific relationship to

              Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-24680 Some themes break when due to hidden topics

              • Minor - Minor loss of function where workaround is possible
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                Hide
                Permalink
                Dan Poltawski added a comment -

                Thanks for the report!

                It looks like topics format has previously been fixed, but i've fixed weeks format in PULL-89

                Show
                Dan Poltawski added a comment - Thanks for the report! It looks like topics format has previously been fixed, but i've fixed weeks format in PULL-89
                Hide
                Permalink
                Helen Foster added a comment -

                Thanks Michael and Dan.

                Show
                Helen Foster added a comment - Thanks Michael and Dan.

                  People

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

                    Dates

                    • Created:
                      Updated:
                      Resolved: