sindresorhus/eslint-plugin-unicorn

Rule proposal: Error when adding a class that begins with a dot`classList.add('.`

Chiusa

#1837 aperta il 30 mag 2022

Vedi su GitHubGuida contributor
 (4 commenti) (8 reazioni) (0 assegnatari)JavaScript (468 fork)user submission
help wantednew rule

Metriche repository

Star
 (5022 stelle)
Metriche merge PR
 (Merge medio 4h 30m) (26 PR mergiate in 30 g)

Descrizione

Description

Sometimes, we incorrectly try to add a class that begins with a dot, but linters and indeed browsers do not indicate that this is an error.

The error occurs because we use a dot in selectors for classname, but the dot is not part of the actual class name.

Fail

node.classList.add('.table-of-contents');

Pass

node.classList.add('table-of-contents')

Additional Info

No response

Guida contributor

Direzione di ricerca
Esamina le regole esistenti in eslint plugin unicorn (ad es. 'no array for each') per il pattern. Implementa una regola che rilevi le chiamate `classList.add` in cui l'argomento è un letterale stringa che inizia con un punto. La regola dovrebbe segnalare un errore e suggerire di rimuovere il punto. Considera l'uso di selettori AST o la visita dei nodi CallExpression.
Tech stack
javascript
Dominio
tooling
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
JavaScriptESLint
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80