Community feedback

• 

  Bhuvnesh Upadhayay•2,000

  @bhuvi819381Posted 2 months ago

  Code Review Feedback

  Hello,

  I checked your code, and it’s perfect! Great job! 😄

  Here are a few suggestions to make it even better:

  1. Use at least one <h1> tag
     Every page should have a single <h1> to improve structure and accessibility.

  2. Avoid using px for units
     Use relative units like rem or em for better scalability and responsiveness.

  3. Add some top margin to the <p> tag
     This will improve the spacing and make the layout look cleaner.

  Other than these, everything looks fantastic. Keep up the great work!

  Marked as helpful

  1
• 

  Marzia Jalili•3,970

  @MarziaJaliliPosted 2 months ago

  Nicely done!!!,

  The solution looks AWESOME!!!

  For this layout, you could use grid instead of flexbox.

  Two mean reasons:

  1. First, the size every child element takes is the same, isn't it?
  2. Your code will look much cleaner, just the TWO lines below, man!

  display: grid;
  place-items: center;
  

  Hope this improves your next projects😎

  Marked as helpful

  1