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

sunnyside agency landing page using grid

Vjekoslav 485

@Vj3ko

Desktop design screenshot for the Sunnyside agency landing page coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


any feedback is welcomed :)

Community feedback

P
David Turner 4,110

@brodiewebdt

Posted

You did a great job on this. Looks really good. You want to wrap all of your content in a Main tag. You just have one section named main. Add aria-label="Facebook link" etc.. to the A tag surrounding your font-awesome links. Add Alt text to your two testimonial images. They can stay empty as they are decoration only.

Download AXE DevTools and you can clear accessibility warnings while you code. https://www.deque.com/axe/devtools/

Hope this helps.

Marked as helpful

0

Vanza Setia 27,835

@vanzasetia

Posted

@brodiewebdt There's no need to put the word "link" as long as it has been wrapped with an anchor tag. The same logic applies to the alternative text on img.

Marked as helpful

1
Vjekoslav 485

@Vj3ko

Posted

@brodiewebdt totally forgot to wrap all content in MAIN tag, tnx for reminding me tho, also nice addon to have ( AXE devtools). I installed it, it's quite helpfull.

1

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