issue-24/CardMainMobile #36
Conversation
|
✔️ Deploy Preview for wineberry ready! 🔨 Explore the source changes: ca00db9 🔍 Inspect the deploy log: https://app.netlify.com/sites/wineberry/deploys/6147b9673d41af0008638215 😎 Browse the preview: https://deploy-preview-36--wineberry.netlify.app |
| export const cards ={ | ||
| title: "El nuevo vino de arandanos", | ||
| description: "Tal y como su nombre indica, el vino de arándanos se elabora a partir de los arándanos. La piel y la carne de la planta se fermentan por completo durante algún tiempo para obtener el vino de arándanos. No sólo es un vino sabroso, sino que tambien ofrece varios beneficios para la salud.", | ||
| paragraph: [ |
There was a problem hiding this comment.
Holis! Como pequeño cambiecito, acá podríamos poner "paragraphs" en plural, porque son muchos párrafos. Y así mismo en la línea 1, le pusiste export const "cards", y en realidad es una sola card.. Ahí debería ir en singular jiji.
| import image from '../../assets/imagen-vindano-1.png'; | ||
| import Button from "../Button"; | ||
|
|
||
| function CardMainMobile() { |
There was a problem hiding this comment.
Muy buen trabajo! Está hermoso el componente!! Sí una cosita!! Actualizate esta rama con master (desde la rama, git pull origin master) y ahí te van a llegar los cambios de rutas que trabajó Cel.
Esta card, iría en la ruta /lo-nuevo que ya la tenemos lista!
Si entrás a esa url de la app te va a mostrar esto:
La idea sería que pongas tu componente ya ahí dentro de esa ruta. Sería acá:
Borrale las lineas comentadas, ese style con margin también, y dentro del div, donde dice "news work", borrá eso y poné tu componente ahí, así ya lo tendremos visible en master!
Luego en otra tarea trabajaremos que en Desktop se renderice la otra card que corresponde a Desktop. Pero de esta forma ya nos quedaría puesta tu card en la app :D
| &--title { | ||
| font-family: $raleway; | ||
| font-style: normal; | ||
| font-weight: bold; |
There was a problem hiding this comment.
No hace falta poner font-family ya que eso ya esta definido en _base, cambiara font-weight: bold por 700 como dice figma!
… into issue-24/CardMain
…onente en su respectiva ruta
| letter-spacing: 0.05em; | ||
| align-items: center; | ||
| text-align: center; | ||
| letter-spacing: 0.05em; |
There was a problem hiding this comment.
como otro pequeño detalle letter-spacing esta definido dos veces
| &--paragraph { | ||
| font-family: $raleway; | ||
| font-style: normal; | ||
| font-weight: normal; |
There was a problem hiding this comment.
tambien cambiara por el font-weight que dice figma (400)
Descripcion
Trabaje en el componente mobile al que direcciona el boton de Main que dice "Conocé lo nuevo"
Capturas de pantalla
Issue
#24