43081j / 43081j/postcss-styled-components
Declarations that use interpolations with Props seem to be ignored
- 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