Four Card Feature

Community feedback

• 

  Szymon Rojek•4,540

  @SzymonRojekPosted almost 4 years ago

  Hi Vaishnav,

  Well done :D

  A few tips below:

  • instead of div class="header", use the header tag;
  • we can use only one the h1 on a page, in this project you can create one h1 with two spans inside main-heading: Reliable, efficient delivery and sub-heading: Powered by Technology;
  • instead of div class="container" you can use the main tag just to indicate the main content of a page;
  • alt text => In this project, images have an only a decorative role - that's a reason why alt text should be provided as an empty (alt="") so img can be ignored by assistive technologies, such as screen readers;
  • div class="card-icon", div class="card card-four" are unnecessary;
  • check a project in your browser by the inspector on different devices => what's about the tablet breakpoint (two rows with two boxes each). In my opinion one column till 1200px is too long, check it out;
  • add max-width to the container;

  Ps. Don't forget to upvote any comments on here that you find helpful.

  Greetings :D

  0