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] Mauro Monereo #57

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

mmonereo
Copy link

Faltan miles de cosas pero creo que el layout está medio decente y tiene un poco de responsive.

@mmonereo mmonereo changed the title [MAD-FT-1021] Mauro Monereo [MAD-DE] Mauro Monereo Oct 19, 2021
@mmonereo mmonereo changed the title [MAD-DE] Mauro Monereo [MAD DE] Mauro Monereo Oct 19, 2021
@ta-web-mad
Copy link
Owner

Hola Mauro!

"Faltan miles de cosas" no es nada representativo de tu trabajo 😆 Está muy bien!

Da gusto ver tu código, tan organizadito. Te diría de poner algún espacio entre cada apartado o sección, por darle mayor legibilidad, y utilizar las etiquetas section y article, en vez de todo divs, ya que le da más sentido y facilita la lectura. Por ejemplo, en la sección de fondo rojo de los círculos, eso podría ser una section y cada uno de los bloques internos un article.

Por decirte alguna cosilla más, cuando vas a usar algo muchas veces en CSS, por ejemplo el color de letra blanco, en vez de poner en cada clase el 'color: white' puedes también crear una clase que sea 'white-text' y aplicarla siempre que quieras eso concreto, aunque esto ya es más una cuestión de estilos/preferencias.

Sigue así! 🔥

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