Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MA-9]: Fixed the keyboard interactivity of tablist and added appropriate ARIA properties #13

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ayush-chauhan233
Copy link
Collaborator

@ayush-chauhan233 ayush-chauhan233 commented Dec 9, 2024

Summary

  • Updated the tab list structure,
  • Updated the logic to navigate between tabs using the keyboard.
  • Added appropriate ARIA properties to the tab and tab panels.

Steps to reproduce

  • Navigate to the Display tab controls using arrow keys.
  • Notice that the tab controls are not operable using arrow keys.

Expected Behavior

  • There should be a single tab list that can contain multiple groups containing their own tabs, and all the tabs must be keyboard accessible.

Ticket Link

Jira https://mattermost.atlassian.net/browse/MM-61588

Screenshots

Before After

Release Note

NONE

{this.props.pluginTabs.map((tab, index) => this.renderTab(tab, index))}
<div
key={'plugin preferences heading'}
role='heading'

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

don't we need to add aria-level here ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants