sindresorhus/eslint-plugin-unicorn

Rule proposal: `prefer-scoped-selector`

Chiusa

#1429 aperta il 16 lug 2021

Vedi su GitHubGuida contributor
 (5 commenti) (2 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

Many people don't know the Element.querySelector() and Element.querySelectorAll() behaves differently than most common JavaScript DOM libraries, details.

Got the idea here

Fail

const inner = element.querySelectorAll('.outer .inner');

Pass

const inner = document.querySelectorAll('.outer .inner');

const inner = element.querySelectorAll(':scope .outer .inner');

I'm not sure about these, they are never going to get wrong result.

const inner = element.querySelectorAll('.a');

const inner = element.querySelectorAll('.a + .b');

Should we enforce them consistently, or only check descendant combinator and child combinator?

Guida contributor

Direzione di ricerca
Per prima cosa, leggi il commento dell'issue collegato (#1404) e comprendi le note dell'utente su Element.querySelectorAll. Quindi, esplora la directory delle regole del repository eslint plugin unicorn per vedere le implementazioni esistenti delle regole per i pattern. La decisione principale è se la regola debba segnalare qualsiasi selettore che inizia con un combinatore discendente o figlio quando chiamato su un elemento, o solo quando il selettore ha uno spazio iniziale (discendente). Rivedi le discussioni nei commenti dell'issue per valutare la preferenza dei manutentori. Implementa la regola in un nuovo file sotto rules/, aggiungi test e aggiorna la documentazione. Concentrati sul rilevamento AST di CallExpression con la proprietà callee 'querySelector' o 'querySelectorAll' e un argomento TemplateLiteral o Literal.
Tech stack
javascript
Dominio
frontend
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.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
ESLint plugin developmentJavaScript ASTCSS selectors
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40