Uploaded image for project: 'Sakai'
  1. Sakai
  2. SAK-22747 Accessibility: Keyboard interactivity bugs with the neoportal worksites menu
  3. SAK-22748

Accessibility: Tabbing of of the "X" close icon in the more sites drawer closes the drawer

    XMLWordPrintable

    Details

    • Type: Sub-task
    • Status: CLOSED
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 2.9.0
    • Fix Version/s: 2.9.0, 10.0
    • Component/s: Accessibility, neoPortal
    • Labels:
      None
    • Environment:
      This issue occurs in Trunk:
      sakai/trunk on Oracle - Built: 10/05/12 08:00 - Sakai Revision: 114039 (Kernel 1.4.0-SNAPSHOT)- Server sakai-nightly.uits.iupui.edu

      Description

      The more sites drawer in the neoPortal worksites menu closes when a user attempts to use a keyboard to move focus off of the "X" close drawer icon.

      Why it's an issue:
      The keyboard only user and screen-reader user should be able to navigate to and explore all of the controls in the more sites drawer. One common technique used by screen-reader users to learn about the available controls is by tabbing through them. The screen-reader user needs to be able to cycle through all of the options in a modal dialog simply by tabbing through them. Controls should never be activated simply by tabbing onto the control or by tabbing off of the control.

      Solution:
      The only keyboard interaction that should activate the close icon to close the drawer is when the enter key is pressed while the close icon has focused. This will happen by default, as it is the default browser operation to activate the onclick handler on links and buttons when the enter key is pressed on them.

      The following piece of code should be removed:

      jQuery('#otherSitesMenu a').last().keydown(function (e) {
      if(e.keyCode == 9 && !e.shiftKey)

      { closeDrawer(); }

      });

        Gliffy Diagrams

          Zeplin

            Attachments

              Activity

                People

                Assignee:
                a.fish@lancaster.ac.uk Adrian Fish
                Reporter:
                brichwin Brian Richwine
                Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                  Dates

                  Created:
                  Updated:
                  Resolved:

                    Git Integration