Simple card using html and css

Community feedback

• 

  Abdul•8,540

  @SamadeenPosted over 2 years ago

  Salam Omar!.. Cheers on completing the challenge.. I'd suggest you scale up your container a little then you'd be good to go.. Over all you did well. Secondly.. You might want to name your div without using the dash(-).. underscore(_) is better

  Happy Coding!!!!

  Marked as helpful

  0
• Account deleted

  Hello there! 👋

  Congratulations on finishing your challenge! 🎉

  I have some feedback on this solution:

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

  • headings should only increase by one use h1 first then h2 etc..

  • you can make it responsive by decreasing the container width on small screens using @media queries

  i hope this is helpful and goodluck

  Marked as helpful

  0

  ---

  

  Omar•160

  @omarsaleh1Posted over 2 years ago

  @Old1337 Thanks, man!

  0
• 

  Naveen Gumaste•10,480

  @NaveenGumastePosted over 2 years ago

  Hello omar ! Congo 👏 on completing this challenge

  Let's look at some of your issues, shall we:

  • Learn more on accessibility issues

  • Add Main tag after body <main class="container"></main>

  • Headings should only increase by one use h1 first then h2 etc

  happy Coding😀

  0