43081j / 43081j/postcss-styled-components

Declarations that use interpolations with Props seem to be ignored

Aperta
#8 4 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
TypeScript
Stelle
12
Fork
1
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

Emotion allows changing styles based on Props ( https://emotion.sh/docs/styled#changing-based-on-props ).
Right now declarations using this synthax seem to be ignored by the linter.

I think that for example the following cases should result in an error:

Example 1:
https://stylelint.io/user-guide/rules/declaration-block-no-duplicate-properties

```
const Button = styled.button`
color: ${props => (props.primary ? 'hotpink' : 'turquoise')};
color: ${props => (props.primary ? 'hotpink' : 'turquoise')};
`
```

Example 2:
https://stylelint.io/user-guide/rules/color-no-invalid-hex

```
const Button = styled.button`
color: ${(props) => (props.primary ? '#f00' : '#00')};
`;
```

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Direzione di ricerca

La issue riguarda il fatto che il linter ignora le dichiarazioni di stile che usano interpolazioni con Props in styled-components. Esamina il parser nel plugin di sintassi postcss-styled-components per vedere come gestisce attualmente le interpolazioni dei template literal. Dovrebbero essere aggiunti casi di test per verificare che vengano rilevate le regole relative alle proprietà duplicate e ai colori esadecimali non validi quando si usano Props. Controlla la test suite per trovare esempi esistenti di parsing di styled-components.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Ambito
devtools, testing-qa
Tipo di issue
Bug
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Specificata chiaramente
Idoneità per principianti
55/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.