Stas Prview Card Component with React, Styled-Components and CSS Grid

Community feedback

• 

  バレンタイン 😈•64,190

  @VCaramesPosted about 2 years ago

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

  • To not only improve your HTML’s code but to identify the main content of you page, you will want to wrap your entire component inside the <Main> Element.

  • The only heading in this component is the “Get insights that help your business grow” everything else will be wrapped in a Paragraph Element.

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

  • The image serve no other purpose than to be decorative; It adds no value. The Alt Tag should left blank and have an aria-hidden=“true” to hides it from assistive technology.

  • 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, feel free to reach out to me.

  Happy Coding! 🍂🦃

  Marked as helpful

  1
• 

  Mahmoud•150

  @MamoutyPosted about 2 years ago

  Thanks for the information. Could you please explain to me when an image should have a description for assistive technology ?

  0