Closed Bug 1786065 Opened 2 years ago Closed 2 years ago

Add aria-label and aria-labeled-by to Spotlight modal

Categories

(Firefox :: Messaging System, enhancement, P1)

enhancement

Tracking

()

RESOLVED INVALID
Iteration:
106.1 - Aug 22 - Sept 2

People

(Reporter: mviar, Assigned: nsauermann)

References

(Blocks 1 open bug)

Details

Feedback from accessibility team suggests we give Spotlight modals a title using 'aria-label' and 'aria-label-by', with the modal labeled by the header.

Assignee: nobody → lsmith
Assignee: lsmith → nsauermann
Iteration: --- → 106.1 - Aug 22 - Sept 2

(In reply to Meg Viar from comment #0)

Feedback from accessibility team suggests we give Spotlight modals a title using 'aria-label' and 'aria-label-by', with the modal labeled by the header.

I might be misunderstanding the code or missing context but it seems like there already is an aria-labelledby on the main for the Spotlight modal. Where the label is the content (h1) and it has a role of dialog. I tested it with a screen reader and it is reading the Title and then you are currently in a dialog, to interact with the dialog...

<main class="screen AW_CHOOSE_COLORWAY  " role="dialog" pos="split" tabindex="-1" aria-labelledby="mainContentHeader">
Flags: needinfo?(mviar)
Priority: -- → P1

@Negin, looks like we're covered. Thanks for investigating.

Flags: needinfo?(mviar)
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.