Inheritence

Community feedback

• Account deleted

  Hello there! 👋

  Congratulations on finishing your challenge! 🎉

  I have some feedback on this solution:

  • its probably because the container too small for the text on small screens you can use @media queries and change the width of the container to 80% .

  • Always Use Semantic HTML instead of div like <main> <header> , etc for more info.

  • Consider using the h1 element as a top-level heading only (all h1 elements are treated as top-level headings by many screen readers and other tools) like for example use h1 first then h2 etc.

  • Images must have alternate text

  if my solution has helped you do not forget to mark this as helpful!

  Marked as helpful

  1
• 

  Mubeen Shaikh•160

  @MubeenAhmadShaikhPosted over 2 years ago

  Looks good I have following feedbacks for you,

  • you can use flexbox instead of margin:auto for aligning the card in center. You can refer this Flexbox guide
  • And for responsive site you can refer Beginner's guide to media queries

  If you get stuck checkout my solution. Hope this helps you

  1
• 

  Hexer•3,660

  @EmmanuelHexerPosted over 2 years ago

  Great work overall man. Keep it up

  1