Four card feature section using scss and flexbox

Community feedback

• 

  Adarsh Pratap•5,515

  @adarshcodesPosted over 4 years ago

  Hey! @Alex F, Great work on this solution👍 And congratulation on completing your first solution of the frontend mentor. Let's see your solution from a few aspects

  1. Design aspect

  • Your design looks great, and also like the animation on those cards😍.
  • You can slightly increase the font-size of the heading.

  2. Responsive view

  • As you can and I can see in your solution, there is a few problems with the layout of the cards in a responsive view, you can remove flex-wrap: wrap and add manual layout using flexbox for that purpose.
  • Didn't find any issue at 1140px(can you explain🙂)

  3. Code aspect

  • Your code looks fine.

  4. Issues aroused

  • You can solve the HTML issue.

  Keep Coding🐱‍💻.

  2

  ---

  

  Alex•125

  @SignetOHaraPosted over 4 years ago

  @adarshcodes Thank you very much, really appreciate the feedback! I will try to improve on my code :) I suspect your suggestion of removing flex-wrap will solve the issue I was having. Will update this page once I have done all the above.

  0

  

  Alex•125

  @SignetOHaraPosted over 4 years ago

  @adarshcodes Thanks for the help again. Cleaned up the code so hopefully more responsive, better heading and also resolved the html validation issues. Onto project number 2!

  1