jsx-eslint/eslint-plugin-react

Rule proposal: forbid innerHTML prop on DOM element tags

Aperta

#712 aperta il 26 lug 2016

Vedi su GitHubGuida contributor
 (1 commento) (0 reazioni) (0 assegnatari)JavaScript (2731 fork)batch import
help wantednew rule

Metriche repository

Star
 (9293 stelle)
Metriche merge PR
 (Metriche PR in attesa)

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