Flexbox

Community feedback

• 

  Mojtaba Mosavi•3,760

  @MojtabaMosaviPosted over 3 years ago

  Hi!, you've done great job on the desktop and mobile view. Here some things to improve

  1- Regarding your markup, for the card's heading I would instead of a h1 use a h2 because h1 semantic meaning is the main heading of a page.

  2- Upon resizing the content on the pages shrinks and grows with make the placement of the button look a bit off, I would've fixed this with setting a min-width on card.

  Keep coding :)

  Marked as helpful

  2

  ---

  

  Vinam•120

  @Vinam09Posted over 3 years ago

  @MojtabaMosavi Hey there, Thanks for the feedback. I appreciate it. I will keep it in mind for future challenges. Thanks again!

  0