Skip to content
Submitted almost 3 years ago

Stats

Orincy Whyte•20
@oswhytecodes
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Stats preview card component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

There is a lot more I could have done. I started this right as my JS course started, so I went back to it almost a month later 😬 and was just staring at my own code like I witnessed a m*rder. My biggest issue was the photo kept disappearing after I added the media query for small devices. I whipped up something so it’s hanging on by a string. All advice is more than welcome.

Community feedback

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

    One tip is to use descriptive class names so that if you look at your code later it will make sense. You also don't have an overlay on the image.

    Wrap your main content in a Main tag and it will help clear the accessibility warnings. You also want to use an H1 tag whenever possible. Change the H2 to an H1.

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

    Hope this helps.

    Marked as helpful
    1
    Orincy Whyte•20
    @oswhytecodesPosted almost 3 years ago

    @brodiewebdt whoaa thanks !!! I appreciate it

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

    Your welcome. Glad I could help.

    1

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