Skip to content

ContextualMenu aria label is placed incorrectly, should be placed on the item with role="menu" instead of the containing ul element #1693

Description

@petdun2519

Priorities and help requested (not applicable if asking question):

Are you willing to submit a PR to fix? Yes #1694
Requested priority: Normal
Products/sites affected: VSTS

Describe the issue:

Vendors have filed bugs saying that the role="menu" div is missing a name property, aka an aria-label, describing what that menu is (the current pattern for focus is setting focus onto the menu div not an individual menu items, so an aria-label there provides context)

WIT id # 984709 # 976804

Actual behavior:

It is on the surrounding ul which i don't think ever gets focused

Expected behavior:

Would be on the role="menu" element

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Fields

    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions