Stats Preview Card Using Bootstrap 5

Community feedback

• 

  Sara Dunlop•430

  @RiscloverPosted about 3 years ago

  Hi! I know you just posted, but I had a few comments for you.

  First of all, good job on your page so far! Just a few things I've noticed:

  • You've used the wrong font for your stats (the bolded stats, like "10k+" and such).
  • You didn't try to change the image color. I just completed this challenge as well, and if you'd like a hint, I was able to do something with color by changing the image's opacity. I know there are different ways to do this, and I'm not sure if mine is even the "correct" or suggested way, but yeah.
  • A big part of this challenge was to make the page responsive so that there's a "desktop" look and a "mobile" look. You've got the desktop look down pretty well, but I noticed that you didn't try to make it responsive and match their mobile screenshot? So that's something you can work on.
  • One last thing: According to the HTML Semantics Report, you've used width='100%' in this line of code: <img src=`images/image-header-desktop.jpg` alt=`image-header-desktop` width=`100%`> I don't think you're supposed to use the % sign for inline CSS. Besides, you should try to eliminate all of your inline CSS and practice only using external CSS. :)

  Other than the above, I thought your solution was fairly well done! Good job!

  Marked as helpful

  0

  ---

  

  Muhammad Habib Taqiyullah•40

  @muhabibtaPosted about 3 years ago

  Hi @Risclover,

  Ah, Thanks a lot for your explained feedback :D

  0