Renszo Camacho• 1,615
@RenszCamacho
Posted
Hi, Eric. Well done.
I have been checking your code and I have seen some issues.
The first suggestion is that you do mobile-first. It's easier.
Your layout looks weird. you have the footer in the middle of the screen. I think you can place all the items, just with flex-box. Although I have seen that you have used absolute position.
Your images are broken, it is because you have set the path wrong, and for good practices, it would be great if you don't leave the alt="" empty at least in these cases.
- The alt attribute holds a text description of the image, which isn't mandatory but is incredibly useful for accessibility — screen readers read this description out to their users so they know what the image means. Alt text is also displayed on the page if the image can't be loaded for some reason: for example, network errors, content blocking, or linkrot.
This how Id solve it.
<img src="./images/image-colton.jpg" alt="Colton Smith" />
Hopefully, it helps.
Happy coding, mate 🧑💻
2
