andreypopp / andreypopp/react-css-components
Fix react unkown props
Abierto
help wanted
- Lenguaje dominante
- JavaScript
- Estrellas
- 672
- Forks
- 14
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
when using the following syntax with React `15.x`
```css
Label {}
Label:prop(mode == 'edit') {}
```
```jsx
```
React throws the following error:
```
Warning: Unknown prop `mode` on
tag. Remove this prop from the element. For details, see https://fb.me/react-unknown-prop
```
If I understand correctly what is being said [here](https://gist.github.com/jimfb/d99e0678e9da715ccf6454961ef04d1b), you should be able to remove unknown props. Basically, that means that the syntax you are currently using should be deprecated.
I would suggest keeping to the `variant` prop:
```jsx
```
If I understand correctly what is being said [here](https://gist.github.com/jimfb/d99e0678e9da715ccf6454961ef04d1b), you should be able to remove unknown props. Basically, that means that the syntax you are currently using should be deprecated.
I would suggest keeping to the `variant` prop:
```jsx
```
Tell me what you think.
**EDIT:** On second thoughts, the syntax is still very useful in many cases. For instance, you could choose to change the style of the component when `onClick` prop is set. I think it should be clearer in the doc that the library does not whitelist the props, and that you should use this feature only with props the component accepts.
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Evaluación
Este issue todavía no se ha evaluado.