Mobile first responsive design CSS Flexbox, JavaScript, HTML

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted almost 4 years ago

  Hey, Dexter Dick! 👋

  Nice to see you complete another challenge! I think you've done a good job on this one! 👏

  Your solution looks good and is responsive. 👍 I only suggest using the <button> element (rather than the anchor tag) for the button of the card component to make your HTML a little more semantic. 😉

  Keep coding (and happy coding, too)! 😁

  0

  ---

  

  P

  Dexter Dick•185

  @DexterDickPosted almost 4 years ago

  Thanks @ApplePieGiraffe you are right that makes more sense. Thanks -:)

  0
• 

  Web Frog•1,840

  @MasterDev333Posted almost 4 years ago

  Great work @Dexter. Your solution is spot on. Fully responsive and close to design. It would be better if you set border-radius and overflow: hidden to .component and rather than setting border-radius individually to child elements. You can see some white spaces at the edge of .component. Hope it helps. Happy coding :)

  0

  ---

  

  P

  Dexter Dick•185

  @DexterDickPosted almost 4 years ago

  Thanks @MasterDev333 I implemented border-radius on .component and set overflow to hidden. I was wondering why I was getting the white space at the edge of the component.

  Your feedback is highly appreciated. Thank you.

  0