Skip to content

MM-61616: Add tab roles inside Emoji Picker#37

Open
ayush-chauhan233 wants to merge 4 commits intomasterfrom
MM-61616
Open

MM-61616: Add tab roles inside Emoji Picker#37
ayush-chauhan233 wants to merge 4 commits intomasterfrom
MM-61616

Conversation

@ayush-chauhan233
Copy link

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

Summary

There are tabs without appropriate role, and state information. Examples include:
-Recently Used, Smileys & Emotion, People & Body, Animals & Nature, Food & Drink, Travel Places, Activities, Objects, Symbols, Flags and Custom tab controls

Steps to reproduce

  • Navigate to Recently Used, Smileys & Emotion, People & Body, Animals & Nature, Food & Drink, Travel Places, Activities, Objects, Symbols, Flags and Custom tab controls.

  • Inspect the element.

  • Notice that the Recently Used, Smileys & Emotion, People & Body, Animals & Nature, Food & Drink, Travel Places, Activities, Objects, Symbols, Flags and Custom tab controls are incorrectly coded as a link.

Note: Due to the structure of the component, we couldn't comply with accessibility requirements completely.

Ticket Link

https://mattermost.atlassian.net/browse/MM-61616

Screenshots

before after
image image

Release Note

NONE

Choose a reason for hiding this comment

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

Suggested change
role='tab'
role='tab'
type='button'

Choose a reason for hiding this comment

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

aria-orientation missing

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