Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

html, css, vanilla JavaScript

Jay Patel 225

@jaypatel9800

Desktop design screenshot for the Agency landing page coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


Is there any suggestions, please let me know..

Community feedback

Faris P 2,810

@FarisPalayi

Posted

Good job completing this challenge👍 Looks good, especially in the mobile👏

Some of my suggestions are:

  • 'Learn more' should be an interactive element like <a> or <button> depending on what does it do.(Usually, it's an anchor tag)
  • Also, try to add some :focus state styles because it can improve keyboard ⌨ users experience drastically
  • Wrap those social media icons in <a> tags, since those are links to other webpages. And give it an appropriate aria-label or title or screen reader only text for better accessibility.

That's all from me. Have fun coding ✨

Marked as helpful

0

Please log in to post a comment

Log in with GitHub
Discord logo

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