stats-preview-card-component-main

Community feedback

• 

  バレンタイン 😈•64,170

  @VCaramesPosted about 2 years ago

  Hey there! 👋 Here are some suggestions to help improve your code:

  • The purpose of the Main Element is to identify the main content of your page. It is not the container of you component. After the main element, you want add a container to wrap you separate components in.

  • The statistics component is a list of statistics, so it should be built using an Unordered List along with a List Items Element.

  • Increase 📈 the font-size of the statistic numbers to better match the FEM design.

  • To get the image to look like the FEM example, you are going to want to use the Mix-Blend-Mode along with the Multiply Value and include a Opacity with the value of 0.8.

  Code:

  img {
    opacity: 0.8;
    mix-blend-mode: multiply;
   
  }
  

  If you have any questions or need further clarification, let me know.

  Happy Coding! 👻🎃

  Marked as helpful

  0

  ---

  

  Bekzat•480

  @BekEsalyPosted about 2 years ago

  @vcarames thanks a lot for your detailed explanation. I appreciate it. It's so important to me

  0
• 

  Adriano•38,390

  @AdrianoEscarabotePosted about 2 years ago

  Hi Bek Esaly, how are you?

  I really liked the result of your project, but I have some tips that I think you will enjoy:

  • Documents must have <title> element to aid in navigation
  • to improve the design of the page in higher resolutions, we can do this:

  .container {
      margin: 0 auto;
  }
  

  The rest is great!

  I hope it helps... 👍

  0

  ---

  

  Bekzat•480

  @BekEsalyPosted about 2 years ago

  @AdrianoEscarabote thanks a lot for your feedback. Your advice is helpful.

  1

  

  Adriano•38,390

  @AdrianoEscarabotePosted about 2 years ago

  @BekEsaly no problem, keep coding!!

  Marked as helpful

  0