Stat card (html sass), images overlay

Community feedback

• 

  Abdul•8,540

  @SamadeenPosted over 2 years ago

  Hey!! Cheers 🥂 on completing this challenge.. .

  Here are my suggestions..

  • You should use <main class="container card"> instead of <div class="container card">.

  This should fix most of your accessibility issues

  . Regardless you did amazing... hope you find this useful... Happy coding!!!

  Marked as helpful

  0
• 

  Adam Wozniak•230

  @adamwozherePosted over 2 years ago

  Looks good! just a couple ideas; with that tricky background, you could try making the background size a fixed scale, although I think it looks quite interesting how it changes with the screen width!

  You could also put a slightly smaller max-width on the container to match the desktop view, and a bit of padding on the body so it will shrink it a bit when it goes down to mobile view.

  0