Stats preview card component - flexbox, css variables

Community feedback

• 

  Ahmad Syafiq Aqil Wafi•170

  @SyafiqjosPosted about 3 years ago

  Hi @ACdev27,

  Congratulations on finishing the challenge!

  I checked the live website, it's working great and responsive but I have some opinion about them.

  1. The card is seems to be really wide on max-width of 757px. The user who read the text probably got tired to read all the long from left to right. Maybe try to shrink it? so the text appear easy to read from top to bottom.

  2. I see that you using manual margin for the stats. It's great but, there is other efficient and fast approach.

  Change from this..

  .stats {
    display: flex;
    justify-content: flex-start;
    margin-top: 18%;;
  }
  .stats div {
    margin-right: 15%;
  }
  .stats div:nth-child(3) {
    margin-right: 0;
  }  
  

  Into this..

  .stats {
    display: flex;
    justify-content: space-between;
    margin-top: 18%;
  }
  .stats div {
  
  }
  .stats div:nth-child(3) {
  
  }  
  

  Also if you still want to use your current approach it's fine, but it's easier to change div:nth-child(3) into div:last-child to make it even more versatile.

  3. Also I think it isn't necessary to put alt in the image, you could leave it empty alt="", since it's not required for us to describe it. Usually alt is used to describe or make a more explanation to the user based on the paragraph or text. But in this challenge the image is only appeared as a background image, there is no relation between the text and stats. It's better that you use role="presentation" or aria-hidden="true" to make screen reader ignore that image.

  4. Last but not least, I think there is slight typo on the queries stats. It's should be "12M+" instead of "12m+".

  Thats all from me, have a good day and keep improving!

  • syafiqjos

  Marked as helpful

  1

  ---

  

  Anthony•355

  @ACdev27Posted about 3 years ago

  @Syafiqjos Thank you for all your helpful feedback. I incorporated several of your suggestions. I decided to not use the justify-content: space-between, because it seemed to add too much space between the stats, and the spacing when using justify-content: flex-start and the margins I added seemed to better match the design in my opinion.

  0