@saravdberg
Latest solutions
Latest comments
- @Gabrieldev-web-coder
Very nice work.
- @sstepaa@Gabrieldev-web-coder
Hola hermano, espero estés bien, aquí mi aporte.
Solo decirte que te apegues a como esta hecho el diseño, hasta en lo más mínimo, esa es la gracia del frontend, efecto de hover y pudiste usar un cursor:pointer; en los botones, por que son elementos interactivos.
Noté que para alinear el componente usaste margin y padding, y eso esta bien para algunas cosas, para el layout es mejor usar flex o grid, por ejemplo, en el elemento body o la etiqueta contenedora de tu contenido principal pudiste usar flex y justify content para centrarlo o usar unidades de viewport. En cambio el margin y el padding pudo servir bien para los elementos de las cartas, para el parrafo, las imagenes y los botones.
Aún así luce bastante bien, sigue estudiando y practicando, sobre responsive design y las bases de css, ten buen dia.
- @adityapawar12@Gabrieldev-web-coder
Hi, maybe you use propities that don't support the browser, or only you do bad things.
For the background try use viewport units, and use the semantic html, and saw you use a div for a container of an image, the container of an img is the <figure> tag, the space of text that doesn't a paragraph, you can use <span>
Search and learn about BEM is a method to add the classes in html to be more readable, remember that others i'll read your code, and the name of your classes is some confuse.
You used also position to align the text and other things, for layout, the better way is use flexbox or grid, only for the img of victor is a good idea use the position, and you write so many code to a simple card compose, try to do reusable classes and do more writing less as possible.
Sounds hard but no one is born knowing, only stay practicing and studing and you will improve, have a nice day.
Marked as helpful - @befflus@Gabrieldev-web-coder
Hi, i hope you're okay.
To align the background without the media queries use viewport units.
all other is okay, be careful with the details of the design, but well done.
Marked as helpful - @Cpinero@Gabrieldev-web-coder
Good work in this challenge, but be careful with the details of the design, like the font size, weight, and family.
All other is good, keep going.
- @aaustt@Gabrieldev-web-coder
El diseño es muy parecido, pero creo que en el breakpoint 600px hay mucho espacio en blanco, cuando en el diseño se mantiene solo un pequeño espacio, yo hubiese puesto la propiedad de Flex hasta llegar a 768px y recién allí adaptarlo a Tablet y ese mismo cambio sirva para 1024px en adelante.
Como usaste max-width creo que hiciste desktop first, ten en cuenta que en cuestión de SEO y accesibilidad lo ideal es que los diseños sean adaptados primero para dispositivos móviles, (mobile first) usando min-width, es una buena practica maquetar primero para móvil.
Pero aún así esta muy bien, sigue codeando hermano.
