Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MAD DE] ALEJANDRO MALLO #47

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

PandaMallo
Copy link

impossible to make the icons right. hard to align items. to many divs :_(

@BitterBaitBite
Copy link

BitterBaitBite commented Aug 11, 2021

El ejercicio está realmente bien, y muy muy completo.

Respecto a la alineación de los contenidos, veo que aunque haya sido complicado lo has resuelto bien, y eso es una buena noticia porque significa que de aquí a dos días lo harás con los ojos cerrados.

Del tema de los div, todos los que utilizas están justificados y bien utilizados, si bien es verdad que siempre te va a beneficiar usar etiquetas con una mayor semántica en vez de

, por ejemplo ,
, , , etc.

Por último, los iconos están casi perfectos, lo único que falta es el borde, así que no te preocupes, que es un pequeño detalle. Para estos bordes, lo ideal sería jugar tanto con la propiedad border, como con border-radius para redondearlo, como con padding para crear algo de espacio entre el icono y el propio borde.

P.D: se te ha pasado la separación de los distintos bloques del footer. Un poco de padding para la próxima 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants