@MelvinAguilar
Posted
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 helpful
