@mattstuddert
Posted
Hey Florian, you've done an awesome job on the challenge. Great work! 👍
Your solution matches up to the design nicely and your mobile layout looks great. The only two small areas I'd recommend reviewing would be:
- When you first switch from mobile to the desktop layout, the text looks very squashed. I'd recommend reducing the size of the image on the left initially to make more space for the text.
- The
alttext in the social link images will be read out to people using screen readers. So I'd recommend adding more descriptive text. For example, "facebook link" could be changed to "Follow us on Facebook". Making that change would add much more context to screen reader users.
Keep up the great work. I hope you enjoyed the challenge! 🙂
@FloPereira75
Posted
@mattstuddert
Thank you for your advice! It motivates me even more! And for the "alt" thank you for the advice, I never really understood if I had to be precise or not haha! Huge thank you, I'm going to make myself a "newbie" challenge in HTML / CSS and then I will attack a challenge with JS!
And yes, this site is great, that's what I needed to be able to really progress, really cool!
@mattstuddert
Posted
@FloPereira75 you're welcome! I look forward to seeing your future solutions!
