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

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

    XMLWordPrintable

    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.

        Attachments

          Activity

            People

            Assignee:
            timhunt Tim Hunt
            Reporter:
            timhunt Tim Hunt
            Peer reviewer:
            Mike Churchward
            Integrator:
            Dan Poltawski
            Tester:
            John Okely
            Participants:
            Component watchers:
            Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Dates

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