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

3_column_pre-view_card

Aatifโ€ข 90

@AatifxShaikh

Desktop design screenshot for the 3-column preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hello, community this is another pre-view card project that I have been doing, to learn more about css flex and how to style layouts. Hope it is up to the mark

Feedback Appreciated.

Community feedback

@MelvinAguilar

Posted

Hi @AatifxShaikh ๐Ÿ‘‹, good job on completing this challenge! ๐ŸŽ‰

Here are some suggestions you might consider:

  • The <h1> element is the main heading on a web page. There should only be one <h1> tag per page. The HTML Section Heading elements (Reference). This case requires that SUVS, SEDANS, and LUXURY be <h2> elements.
  • The Learn More elements should be anchor tags and not buttons because they redirect to another part of the page.
  • The container isn't centered correctly. You can use flexbox to center elements:
body {
  min-height: 100vh;
  display: flex;
  flex-direction: column;
  justify-content: center;
  align-items: center;
}

Also, remove the margin from the container selector.

Links with more information:

I hope those tips will help you.

Good job, and happy coding!

Marked as helpful

0

Aatifโ€ข 90

@AatifxShaikh

Posted

@MelvinAguilar Thank You for the feedback. Will look into the tips and will try to improve more.

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