Sunnyside landing page

Community feedback

• 

  Boots 😺•1,590

  @adityaphasuPosted over 1 year ago

  Hey there!

  Your solution looks nice!!

  🎯 There's some semantic errors though that should be corrected:

  • You dont need a h1 heading for the screen readers because this challenge's H1 is actually the text "WE ARE CREATIVES". It should be like this:

  <h1 class="bigtext">WE ARE CREATIVES</h1>
  

  • The subsequent big texts in the section "Transform your brand" and "Stand out to the right audience" are actually h2s rather than h3s. They should be like this:

  <h2>Transform your brand</h2>
  <h2>Stand out to the right audience</h2>
  

  • In the testimonial section the text "Client Testimonials" as you have named the class, it's a heading so you should replace the div tag with a h2 tag. One more thing is that you shouldnt use uppercase words in the HTML instead use CSS to modify the text like this:

  .testimonial-heading{
  text-transform: uppercase;
  }
  

  Read more about it Here!.

  The solution is good but with semantic HTML it would be perfect! You can read more about the importance of semantics here.

  Happy coding!🕺🏻

  Marked as helpful

  1

  ---

  

  Aanu-TT•150

  @AanuTTPosted over 1 year ago

  Thanks @adityaphasu. I'll work on it

  1
• 

  Victor Ebegbuna•170

  @victorebegbunaPosted over 1 year ago

  Lovely solution. Just needed some alignment adjustment.

  0