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

Breadcrumbs should only have one <nav> tag, and should have an aria label

    Details

    • Testing Instructions:
      Hide
      1. Using Clean theme, view a range of pages in your Moodle site, with as many differnet page layouts as possible. (in particular, some layout files call $OUTPUT->fullheader(), and others call $OUTPUT->navbar, you need at least one of each sort of layout.)
      2. For each one, verify that the navbar (often mis-called the breadcrumbs) are in exactly one <nav> tag, not two nested ones.
      3. Verify that that nav region is aria-labelled. (NVDA does not seem to support this, so you would need to use JAWS to test this properly, or you could just inspect the HTML.)
      4. Repeat this testing using a theme based on base, instead of bootstrapbase.
      Show
      Using Clean theme, view a range of pages in your Moodle site, with as many differnet page layouts as possible. (in particular, some layout files call $OUTPUT->fullheader(), and others call $OUTPUT->navbar, you need at least one of each sort of layout.) For each one, verify that the navbar (often mis-called the breadcrumbs) are in exactly one <nav> tag, not two nested ones. Verify that that nav region is aria-labelled. (NVDA does not seem to support this, so you would need to use JAWS to test this properly, or you could just inspect the HTML.) Repeat this testing using a theme based on base, instead of bootstrapbase.
    • Affected Branches:
      MOODLE_30_STABLE
    • Fixed Branches:
      MOODLE_29_STABLE, MOODLE_30_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      The breadcrumbs currently have two nested <nav> tags, and neither have an aria-label attribute. This was picked up in OU accessibility testing.

        Gliffy Diagrams

          Attachments

            Activity

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  9/May/16