testimonials-grid HTML & CSS

Community feedback

• 

  Szymon Rojek•4,540

  @SzymonRojekPosted almost 4 years ago

  Hi Catuva21,

  Well done with RWD :D

  A few tips below:

  • in my opinion this div class="centro" doesn't have to be here;
  • instead of div class="container" you can use the main tag;
  • instead of section class="section" probably you can use an article;
  • alt text => don't need to use words like picture or image, photo, icons in the alt text as it's already announced as being an image, so you can easily type a person's name;
  • instead of h2 for the text "Verified Graduate" I'd suggest to use the p tag;
  • also, this is a single page component so the h1 can use with the class sr-only set to hidden visible only for the Screen Readers (we have to use headings gradually and do not skip them => outline algorithm);
  • instead of the h3 I'd suggest using the p tag because it is not a heading => bold text doesn't have to indicate on heading;
  • classes are not readable and descriptive so I'd recommend learning BEM naming convention;
  • max-width should have 1440px;
  • check the img border color (according to the project).

  Keep coding :D

  Ps. Please, don't forget to upvote any comments on here that you find helpful.

  Greetings :D

  1