Design comparison
Solution retrospective
It took me few hours to complete Take a look at the live site and code for any suggestions for betterment. Any correction and guidance is welcomed and appreciated.
Community feedback
- @MelvinAguilarPosted 11 months ago
Hello there ๐. Good job on completing the challenge !
I have some suggestions about your code that might interest you.
-
Social media icons should be links with the icon image, allowing users to easily navigate to the respective social media pages.
-
Elements like "FAQs," "Contact Us," "Privacy Policy," "Press Kit," and "Install Guide" are not headers; they are links for navigating through the page. Use <a> inside an unordered list <ul>
-
The
<article>
element is typically used to encapsulate a piece of content that makes sense on its own and can be distributed and reused independently, which might not be the case if it only contains an image. Consider if another container element might be more appropriate. -
Use font-size in rem or em units rather than pixels for better responsiveness.
-
The alt attribute should avoid redundant words like image, photo, or picture, as the
<img>
tag already conveys that information. Also, avoid unnecessary hyphens, and make the alt text readable.
I hope you find it useful! ๐ Above all, the solution you submitted is great!
Happy coding!
Marked as helpful0 -
Please log in to post a comment
Log in with GitHubJoin 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