jsx-eslint/eslint-plugin-react

Rule proposal: forbid innerHTML prop on DOM element tags

Open

#712 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 try to use pass innerHTML prop to a DOM element tag.

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

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

Bad:

<div innerHTML='HTML' />

Good:

<div dangerouslySetInnerHTML={{ __html: 'HTML' }} />

Guida contributor

Direzione di ricerca
Esamina i pattern delle regole ESLint esistenti nel plugin, quindi implementa una regola che rilevi la prop innerHTML sugli elementi DOM controllando i nomi degli attributi JSX e i tipi di elemento. Fai riferimento al codice sorgente di React per il messaggio di avviso e le condizioni.
Tech stack
javascriptreact
Dominio
developer experiencetooling
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
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
ESLintReact
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65