3-Card-Preview using Pure HTML and CSS

Community feedback

• 

  Joseph•770

  @Jos02378Posted about 3 years ago

  Hey @satyamallick7, good job on this solution!

  Some suggestions for you:

  1. You can use text-transform: uppercase to capitalized text in CSS.
  2. Try to wrap your code in the body using a main element to clear some accessibility issues.
  3. You can try to use flexbox to create this solution, it will be easier for you to create the layout for the card.
  4. Don't set a fixed height to an element as it can cause issues. You can use min-height or not setting the height at all so your element can grow as your content grows.
  5. You can try to follow a CSS naming convention called BEM. You can watch this video for more information Watch the video.
  6. You can try to use more relative units like rem and em in the future. Here is a link to an article that explains when to use which unit see the article.

  I hope this helps, good luck!

  Marked as helpful

  2
• 

  Bikrant Malla•990

  @vikrantmallaPosted about 3 years ago

  Add grid in the box you are good to go. learn Grid and flexbox

  Marked as helpful

  0