jsx-eslint/eslint-plugin-react

Rule proposal: disallow children on DOM element tags when contentEditable

Aperta

#713 aperta il 26 lug 2016

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

Metriche repository

Star
 (9294 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

React will issue a warning in dev if you use contentEditable with children. This can be suppressed with suppressContentEditableWarning.

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

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

Bad:

<div contentEditable>
  Children
</div>

Good:

<div contentEditable />

<div contentEditable suppressContentEditableWarning>
  Children
</div>

Guida contributor

Direzione di ricerca
Esamina le regole esistenti di eslint plugin react (ad esempio no danger with children) per comprendere il pattern. Crea una nuova regola che controlli gli elementi DOM con contentEditable e figli. Implementa la regola in un nuovo file sotto lib/rules/. Scrivi test e aggiorna la documentazione.
Tech stack
javascriptreactnodejs
Dominio
frontendtooling
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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
JavaScriptReactESLint
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75