jsx-eslint/eslint-plugin-react

Rule proposal: disallow onFocusIn and onFocusOut on DOM element tags

Open

#714 aperta il 26 lug 2016

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)JavaScript (2797 fork)batch import
help wantednew rule

Metriche repository

Star
 (8630 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

React will issue a warning in dev if you use onFocusIn or onFocusOut.

https://github.com/facebook/react/blob/85dcbf83/src/renderers/dom/shared/ReactDOMComponent.js#L196-L200

This should be easy enough to enforce for most cases with a linter rule.

Bad:

<div onFocusIn={() => {}} />

<div onFocusOut={() => {}} />

Good:

<div onFocus={() => {}} />

<div onBlur={() => {}} />

Guida contributor

Direzione di ricerca
Studia il codice sorgente di eslint plugin react, in particolare una regola esistente come 'no onchange readonly'. Crea un nuovo file di regola per 'no focus in out', scrivi test che corrispondono agli esempi buoni e cattivi, aggiungi documentazione e registra la regola nell'indice del plugin.
Tech stack
javascriptreact
Dominio
frontendweb dev
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitESLintJavaScriptReact
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75