Skip to content
Submitted almost 3 years ago

Responsive Sunnyside attempt

Lorenzo•150
@ingegnerlorenzo
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Agency landing page
  • HTML
  • CSS
  • JS
2junior

Design comparison

SolutionDesign

Solution retrospective

Hey everyone, I think my code (mostly css) is not that clean. Could someone give me some advice on how to make it better? Thanks in advance for the feedbacks on the work! It was challenging yet funny!

Community feedback

  • David Turner•4,150
    @brodiewebdtPosted almost 3 years ago

    I am just getting used to writing more accessible code, and that AXE plugin has been great for that. That's why I recommend it.

    Marked as helpful
    0
  • David Turner•4,150
    @brodiewebdtPosted almost 3 years ago

    Good job on this. This wasn't an easy one.

    Wrap your container div in a Main tag, and change the attribution div to a footer and add aria-label="pinterest link" to the footer a tags and it will clear the accessibility warnings.

    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
  • Lorenzo•150
    @ingegnerlorenzoPosted almost 3 years ago

    Thanks for the advice and for the kind feedback - I definetily want to use way more ARIA labels in HTML in order to make my works more accessible!

    0

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