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

Flexbox;html;css;

@phoenixjmh

Desktop design screenshot for the Product preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

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

@Coltensiemer

Posted

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

Posted

@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
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