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

[Fix] no-unknown-property: add more capture event properties #3402

Merged

Conversation

sjarva
Copy link
Contributor

@sjarva sjarva commented Sep 5, 2022

Closes #3397 by

  • fixing the typo in controlList to controlsList

Closes #3400 by

  • by adding many xxxYyyCapture properties

In addition:

@codecov
Copy link

codecov bot commented Sep 5, 2022

Codecov Report

Merging #3402 (1c0c92b) into master (4be5632) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3402   +/-   ##
=======================================
  Coverage   97.58%   97.58%           
=======================================
  Files         123      123           
  Lines        8958     8958           
  Branches     3270     3270           
=======================================
  Hits         8742     8742           
  Misses        216      216           
Impacted Files Coverage Δ
lib/rules/no-unknown-property.js 98.83% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@ljharb ljharb force-pushed the no-unknown-property-more-capture-properties branch from 95b2ac0 to 1c0c92b Compare September 5, 2022 16:22
@ljharb ljharb changed the title no-unknown-property more capture properties [Fix] no-unknown-property: add more capture event properties Sep 5, 2022
@ljharb ljharb merged commit 1c0c92b into jsx-eslint:master Sep 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

no-unknown-property does not recognize onClickCapture on <div> Missing controlsList property
2 participants