Uploaded image for project: '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
          • Status: Closed
          • Priority: 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

                Attachments

                  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
                    poltawski 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
                    poltawski 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
                    tsala Helen Foster added a comment -

                    Thanks Michael and Dan.

                    Show
                    tsala 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:
                          Fix Release Date:
                          21/Feb/11