@vanzasetia
Posted
Hello there, Ivan! 👋
Nice work on this challenge! 🙌 Your solution is responsive without media queries and looks great! It's amazing that you can handle those background patterns. 👏 When I was doing this challenge I couldn't figure it out by myself. 😅
Anyway, some suggestions from me.
- The photo needs an alternative text so that all users know that there's a photo of Victor Crest. It might also useful for SEO. So, the alternative text for the photo should be Victor Crest. For your information, alternative text for images should not contain any words that related to image (e.g. picture, photo, logo, icon, graphic, avatar, etc). It's already an image element so the screen reader will pronounce it as an image.
- I would highly recommend adding
justify-content: center
and then removing themargin: 0 auto
from the.container
element. Even better, you can remove the.container
element entirely and put thetext-align
property into thebody
element. This way, you can clean up the HTML.
Overall. great work. You manage to handle those background images and finish this challenge without any media queries.. Also, it would be great if you can remove the .container
element because I think you might not need it actually.
I hope this information is useful! Keep up the excellent work! 👍
Marked as helpful