3 Column preview card with vibrant colors.

Community feedback

• 

  Rio Cantre•9,690

  @RioCantrePosted almost 3 years ago

  Hello there! Great job with this challenge. You did well on implementing the design and I would suggest the following for you...

  • Adjust the margin in the rule set of .card__link into margin-top: 3rem;
  • Clean the Whitespaces in your code
  • Wrap the container with specific tag like main for readability
  • Add border radius in the media queries as well

  Overall you did great and Keep it going!

  Marked as helpful

  1

  ---

  

  Aadvik•1,250

  @Aadv1kPosted almost 3 years ago

  @RioCantre Thanks for the review Rio! Appreciate your input. When I started this challenge I was new to css, so all the bad practices. I will definitely prettify and format it and make the html more semantic.

  1
• 

  Naveen Gumaste•10,480

  @NaveenGumastePosted almost 3 years ago

  Hay ! Good Job you made it look nearly perfect to the preview

  These below mentioned tricks will help you remove any Accessibility Issues

  -> Add Main tag after body like it should be your container

  -> For 1st heading or h1 tag, use header tag and then inside the header put your h1 or h2 etc

  -> But use header tag only once in main heading element.

  Keep up the good work!

  1

  ---

  

  Aadvik•1,250

  @Aadv1kPosted almost 3 years ago

  @Crazimonk Appreciate your input Naveen, I will get back to implementing these fixes later, currently I am working on another challenge but thanks for the suggestions!

  1