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

Adds labelling for toggle button v9 and adds a rule for no empty buttons #63

Merged

Conversation

nikitasingla-MSFT
Copy link
Collaborator

@nikitasingla-MSFT nikitasingla-MSFT commented Mar 13, 2024

This PR creates rule for no empty button in v9, and also handles labelling for toggle buttons.

Related work item emptyButtonRule, toggleButton

NOTE: The scenario including icons and button/ToggleButton are already taken care of in this file image-button-missing-aria-v9

tests/lib/rules/toggle-button-needs-labelling-v9.js Outdated Show resolved Hide resolved
docs/rules/toggle-button-needs-labelling-v9.md Outdated Show resolved Hide resolved
docs/rules/toggle-button-needs-labelling-v9.md Outdated Show resolved Hide resolved
tests/lib/rules/toggle-button-needs-labelling-v9.js Outdated Show resolved Hide resolved
tests/lib/rules/toggle-button-needs-labelling-v9.js Outdated Show resolved Hide resolved
@aubreyquinn aubreyquinn changed the title Adds labelling for toggle button v9 Adds labelling for toggle button v9 and adds a rule for no empty buttons Mar 14, 2024
docs/rules/no-empty-buttons-v9.md Outdated Show resolved Hide resolved
lib/rules/no-empty-buttons-v9.js Outdated Show resolved Hide resolved
lib/rules/no-empty-buttons-v9.js Show resolved Hide resolved
@nikitasingla-MSFT nikitasingla-MSFT merged commit f6fe570 into microsoft:main Mar 14, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants