jsx-eslint/eslint-plugin-react

Rule proposal: forbid superfluous enclosing spaces in same-line jsx components

Open

#191 aberto em 17 de ago. de 2015

Ver no GitHub
 (5 comments) (0 reactions) (0 assignees)JavaScript (8.630 stars) (2.797 forks)batch import
help wantednew rule

Description

This would produce a warning:

<span> { tab.title } </span>

as it produces 3 <span> DOM nodes (typically not wanted)

This wouldn't:

<span>{ tab.title }</span>

Guia do colaborador

Pilha de tecnologia
javascriptreactnodejs
Domain
developer experiencetooling
Tipo Issue
feature
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.
half day
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
Basic understanding of ESLint rulesFamiliarity with JSX syntaxJavaScript knowledge
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70
Direção de pesquisa
Explore the existing rules in eslint plugin react (e.g., jsx indent, jsx curly spacing) for patterns on implementing JSX lint rules. Create a new rule file that uses the JSXElement visitor to check for spaces inside JSXExpressionContainer. Specifically, detect when there is a whitespace node between the opening brace and the expression or between the expression and closing brace. Add the rule to the rules index and write test cases including the positive and negative examples from the issue. Consider using eslint plugin react's built in utilities for handling JSX.