Contrib.social

Make the textbox more accessible (unnecessary tabIndex)

Open on Drupal.org →
Created on 4 September 2023, over 1 year ago
Updated 24 September 2023, over 1 year ago

Problem/Motivation

The created widget has a tab index of 0, but also an 'unselectable' attribute. It's not actually possible to tab to it since it has a display: none; set, so its somewhat redundant.

Steps to reproduce

Proposed resolution

jquery-ui already has a similar solution in place.

See https://web.dev/control-focus-with-tabindex/#remove-an-element-from-the-tab-order and https://www.456bereastreet.com/archive/201108/do_not_use_displaynone_to_visually_hide_content_intended_for_screen_readers/

This removes an element from the natural tab order, but the element can still be focused by calling its focus() method.

The tabIndex can be updated to be removed or set to -1 (to be more explicit) so that it's no longer focusable.
The autocomplete dropdown is only ever visible whenever the input field is in focus anyway making it impossible to tab into otherwise making its existence somewhat irrelevant lol.

Remaining tasks

Provide patch/issue fork.

User interface changes

N/A.

📌 Task
Status

Fixed

Version

1.0

Component

User interface

Created by

codebymikey

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024