Skip to content
This repository has been archived by the owner on Jul 15, 2023. It is now read-only.

new a11y rule: no-onchange #278

Closed
HamletDRC opened this issue Sep 27, 2016 · 1 comment
Closed

new a11y rule: no-onchange #278

HamletDRC opened this issue Sep 27, 2016 · 1 comment
Labels
Difficulty: Easy Someone with little to no experience in TSLint should be able to send a pull request for this issue. Domain: Accessibility Rules around accessibility verification, commonly react-a11y-*. Status: Accepting PRs Type: Rule Suggestion Adding a new rule that doesn't yet exist here or in TSLint core.
Milestone
6.0.0-beta

Comments

@HamletDRC
Copy link
Member

no-onchange

Enforce usage of onBlur over onChange on select menus for accessibility.

Also, OnChange event handler should not be used with the select element.
• http://oaa-accessibility.org/wcag20/rule/62/
@JoshuaKGoldberg JoshuaKGoldberg added Status: Accepting PRs Difficulty: Easy Someone with little to no experience in TSLint should be able to send a pull request for this issue. Type: Rule Suggestion Adding a new rule that doesn't yet exist here or in TSLint core. Domain: Accessibility Rules around accessibility verification, commonly react-a11y-*. labels Jul 7, 2018
@sahalsaad
Copy link
Contributor

I'll work on this.

sahalsaad added a commit to sahalsaad/tslint-microsoft-contrib that referenced this issue Oct 12, 2018
@sahalsaad
microsoft#278 Implement react-a11y-no-onchange.
effb095
@sahalsaad sahalsaad mentioned this issue Oct 12, 2018
Merged
sahalsaad added a commit to sahalsaad/tslint-microsoft-contrib that referenced this issue Oct 15, 2018
@sahalsaad
microsoft#278 Fix description & options description.
0aaf6ec
sahalsaad added a commit to sahalsaad/tslint-microsoft-contrib that referenced this issue Oct 15, 2018
@sahalsaad
microsoft#278 Implement react-a11y-no-onchange.
f3b5d8e
JoshuaKGoldberg pushed a commit that referenced this issue Oct 16, 2018
@sahalsaad
#278 Implement react-a11y-no-onchange. (#543)
43b6b3c 
* #278 Implement react-a11y-no-onchange.

* Fix description & options description.

* Remove redundant accessibility.

* Update csv.

* Fix failing test.
@JoshuaKGoldberg JoshuaKGoldberg added this to the 6.0.0 milestone Nov 6, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Difficulty: Easy Someone with little to no experience in TSLint should be able to send a pull request for this issue. Domain: Accessibility Rules around accessibility verification, commonly react-a11y-*. Status: Accepting PRs Type: Rule Suggestion Adding a new rule that doesn't yet exist here or in TSLint core.
Projects
None yet
Milestone
6.0.0-beta
Development

No branches or pull requests
3 participants
@HamletDRC @JoshuaKGoldberg @sahalsaad