43081j / 43081j/postcss-styled-components

Declarations that use interpolations with Props seem to be ignored

Offen
#8 4 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
TypeScript
Sterne
12
Forks
1
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

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')};
`;
```

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Das Issue betrifft, dass der Linter Stildeklarationen ignoriert, die in styled-components Interpolationen mit Props verwenden. Sieh dir den Parser im Syntax-Plugin postcss-styled-components an, um zu sehen, wie er derzeit Interpolationen von Template-Literals verarbeitet. Es sollten Testfälle hinzugefügt werden, um zu überprüfen, dass Regeln für doppelte Properties und ungültige Hex-Farben erkannt werden, wenn Props verwendet werden. Prüfe die Testsuite auf vorhandene Beispiele für das Parsen von styled-components.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Bereich
devtools, testing-qa
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
55/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.