@Miculino
Posted
Congrats on completing the challenge, @dodytrifa
You've done quite a nice job and the responsive design works as expected!
Just a few suggestions based on your final solution that I hope will be useful to you:
-
Increase the size of the buttons just a tiny bit
-
Missing the background color that's in the original design
-
Try to remove the vertical scrollbar. Use absolute positioning for the attribution that's at the bottom of the page. That can help fix the scrollbar situation
-
You should add class names for the other elements as well (h3, p, img)
-
Consider implementing BEM methodology https://en.bem.info/methodology/
-
You can vertically and horizontally center your container by setting flex display on body and using
min-height: 100vh
andalign-items: center
andjustify-content: center
properties
Hope this helps. Keep up the great work!