Volodymyr Ostapyshynā¢ 490
@ostapyshyn
Posted
Hi, Paul! š When changing the width of the page, your content goes beyond the top of the viewport. One way to fix it is to change height: 100vh in the body to min-height: 100vh
Marked as helpful
1
Design comparison
Solution retrospective
Hi Everyone! š
I had some difficulty figuring out what to do at specific viewport sizes but ended up going with what I thought would work the best. Please provide me with some feedback with what I can do better.
šš» Thanks!
Community feedback
Vanza Setiaā¢ 27,795
@vanzasetia
Posted
Hi, Paul! š
Congratulations on finishing this challenge! š
Two suggestions:
- First, alternative text should not contain any words that are related to the word "image" such as logo and icon. The semantic meaning of the
imgelement is already good. - Second, the same logic applies when giving label to link elements. Remove the word "link" from all the
aria-labelfor the social media links.
Assistive technologies such as screen readers will know what those elements are as long as we are using the right HTML elements. š
I hope this helps! Happy coding!
Marked as helpful
1
Please log in to post a comment
Log in with GitHub
Join our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord