@benjoquilario
Posted
Hey @dominiquemc Great job on finishing this one, It looks good and response very well.
Some suggestion:
- If you want a main for the main content on your page you should use the
<main> tag
instead. -I've also been at this place where you want to use the <img> tags instead of using a background image.(feels like it would be more flexible) But I realized, the custom changes you can make with background images are way more than <img> tags...so I would suggest you to use them.
Good luck and keep coding
Marked as helpful
@dominiquemc
Posted
@benjoquilario thank you for reviewing! I'll definitely keep those tips in mind for my next challenge!