sindresorhus/eslint-plugin-unicorn

Rule proposal: `prefer-scoped-selector`

Fechada

#1.429 aberto em 16 de jul. de 2021

Ver no GitHubGuia do colaborador
 (5 comentários) (2 reações) (0 responsável)JavaScript (468 forks)user submission
help wantednew rule

Métricas do repositório

Stars
 (5.022 estrelas)
Métricas de merge de PR
 (Mesclagem média 4h 30m) (26 fundiu PRs em 30d)

Description

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?

Guia do colaborador

Direção de pesquisa
Primeiro, leia o comentário da issue vinculada (#1404) e entenda as observações do usuário sobre Element.querySelectorAll. Em seguida, explore o diretório de regras do repositório eslint plugin unicorn para ver as implementações de regras existentes para padrões. A principal decisão é se a regra deve sinalizar qualquer seletor que comece com um combinador descendente ou filho quando chamado em um elemento, ou apenas quando o seletor tiver um espaço inicial (descendente). Revise as discussões nos comentários da issue para avaliar a preferência dos mantenedores. Implemente a regra em um novo arquivo em rules/, adicione testes e atualize a documentação. Concentre se na detecção AST de CallExpression com a propriedade callee 'querySelector' ou 'querySelectorAll' e um argumento TemplateLiteral ou Literal.
Pilha de tecnologia
javascript
Domain
frontend
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
ESLint plugin developmentJavaScript ASTCSS selectors
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40