Skip to content
Submitted over 1 year ago

Flexbox;html;css;

phoenixjmh•10
@phoenixjmh
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Product preview card component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Flying by the seat of my pants on this one, first challenge. Flex seems useful. Responsive design is tricky for a beginner. Took 3 hours :D

Community feedback

  • Colten Siemer•200
    @ColtensiemerPosted over 1 year ago

    Hey!

    Great work on this project! The solution turned out great.

    On thing I noticed in your html:

    Avoid using more than one <h1> per page.Your second <h1> may be better off as an <h2> or just a <p> . Having more than one <h1> may cause screen readers to jump to all <h1. tags before continuing down the page.

    Check out this resource! https://developer.mozilla.org/en-US/docs/Web/HTML/Element/Heading_Elements

    1
    phoenixjmh•10
    @phoenixjmhPosted over 1 year ago

    @Coltensiemer Hey thanks for the tips I really appreciate it! I may or may not have skipped over alot of the basic HTML fundamentals, assuming what could go wrong..Been feeling that for awhile now haha

    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