Add theme provider #312
Add theme provider #312
Conversation
AlmeAld
commented
Mar 30, 2021
•
AlmeAld
commented
- Cree un objeto theme con la paleta de colores.
- En styles.js del componente HomePage encontraran dos ejemplos para usar theme 🙌.
|
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/openpolitica/votupe/3y9VTiD4Nc2M4mhtxSTKoVaxKwnK |
AlmeAld
requested review from
wixo,
dmarquinah,
gianpietro1,
LuighiV and
paulclindo
| @@ -0,0 +1,55 @@ | |||
| const theme = { | |||
| colors: { | |||
global-styles/theme.js
Outdated
| 300: '#E3F9E1', | ||
| 200: '#F7FDF6', | ||
| }, | ||
| warning300: { |
There was a problem hiding this comment.
ups aquí debería ser sólo warning y error 😅
global-styles/theme.js
Outdated
| 400: '#DEDEDE', | ||
| 300: '#F3F3F3', | ||
| }, | ||
| white: '#FFFFFF', |
There was a problem hiding this comment.
aquí, así como black recomendaría usar otros nombres, porque si luego queremos cambiar black por white, tendríamos que hacer cambio de nombres en el uso. Recomendaría ir con el nombre luminosity donde incluya tanto blanco, negro y escala de grises.
| 500: '#B1B1B1', | ||
| 400: '#DEDEDE', | ||
| 300: '#F3F3F3', | ||
| 200: '#FFFFFF', |
There was a problem hiding this comment.
Perdón por demorarme en llegar a este comentario pero #FFFFFF al ser blanco no debería ser 0?, no me imagino que valor podría tener 100 o 50 en esa nomenclatura. También: negro debería ser 1000?
There was a problem hiding this comment.
@wixo me estoy guiando de nuestra paleta de colores, donde nuestro máximo valor es 700
