stats-preview-card-component flex-box

Community feedback

• 

  Raymart Pamplona•16,100

  @pikapikamartPosted about 3 years ago

  Hey, awesome work on this one. Like what you said, the desktop view is only available right now, but looking at it, it's great.

  Some suggestions on the site would be:

  • Don't use vh unit in the height of the .container as this is not consistent. Try to inspect the layout in dev tools at the bottom, you will notice that the layout is destroyed. It would be better to use rem so that it will be consistent for other users.
  • The .stats could use a ul tag since those are "list" of information about the company website.
  • Also, inside in those list information, those text should not be using a heading tag because they don't really give content on what the section would contain right, so better using p tag on them.
  • Also, avoid using multiple h1 on a page, use only 1 per page okay.

  For now, those only. Still, great job on this one again.

  Marked as helpful

  1

  ---

  

  Julian•280

  @Julr09Posted about 3 years ago

  @pikamart Thank you so much, I will correct those.

  1
• 

  Vicktor•900

  @Victor-NyagudiPosted about 3 years ago

  Hi.

  Good attempt on this one.

  If you're struggling with responsive design, there's an amazing YouTube channel covering CSS run by Kevin Powell you can check out.

  He has a free 21-day course covering responsive layouts that might help you out.

  1

  ---

  

  Julian•280

  @Julr09Posted about 3 years ago

  @Victor-Nyagudi Thank you so much, definitely will check that course.

  0