Frontendmentorproject1

Community feedback

• 

  Rachit Patel•540

  @rachit0706Posted about 2 years ago

  Hey Naitik great attempt of the challenge! But I have few suggestions for you for your code:

  1. You should use a semantic element like main and put every element inside.
  2. You can add a common class like "card" or something else to every card and use flexbox for better layout. Here are some useful links: https://www.w3schools.com/css/css3_flexbox.asp, https://flexbox.tech/
  3. Instead giving different border radius to each card you should give one border radius to the container div and set its overflow property to hidden.

  Marked as helpful

  1

  ---

  

  Naitik Lodha•50

  @naitiklodhaPosted about 2 years ago

  @rachit0706 Thank You for your suggestion Rachit.Your suggestion would help me improve my CSS skills.

  0