Four Card Feature in a diamond layout using flexbox

Community feedback

• 

  Youssef Tadarti•90

  @YoussefTadartiPosted about 2 years ago

  Nice to see you complete another challenge! 😀

  Your solution already looks pretty good. If you're looking to improve your solution, a nice next step you can remove the empty div and use the CSS pseudo-element to create a line over the card . 😊 Just a suggestion! 😉

  Hope this helps. 🙏

  Keep coding (and happy coding, too)! 😁

  Marked as helpful

  0
• 

  Lucas 👾•104,340

  @correlucasPosted about 2 years ago

  👾Hello Noel, congratulations for your new solution!

  Great solution and great code. I saw that you've used main and section and this is already good, but the cards are a big chunk of information and its better give a tag with meaningful markup that describes the content, im this case you can use article replacing the div.

  I saw also that you've used rem for some pieces of code and then px to have a standard its better you use or one or another, this case its better u keep everything with rem.

  👋 I hope this helps you and happy coding!

  Marked as helpful

  0

  ---

  

  Noel•230

  @jayrnoelPosted about 2 years ago

  @correlucas Thank you so much!

  0