Responsive Landing page mainly working with Flexbox

Community feedback

• 

  バレンタイン 😈•64,170

  @VCaramesPosted almost 2 years ago

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

  • The header element should only be wrapping the "logo" and ‘nav`.

  • The ‘alt tag` description in the "logo" needs to be improved upon. The description should state the company name.

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

  • Your headings are being used incorrectly. When it comes to using headings, you are not allowed to skip heading levels. so "Stay productive, wherever you are", "Get early access today" are suppose h2 headings.

  • Your email input and button needs to be wrapped inside a form and have a visibly hidden label attached to it for improved accessibility.

  More Info:📚

  MDN Email Input

  • For the testimonial, it is best to to wrap the testimonial component in a figure element, the individual’s information should be wrapped in a figcaption element and lastly, the testimonial itself should be wrapped in a blockquote element.

  Code:

  <figure>
  	<blockquote></blockquote>
   	<figcaption></figcaption>
  </figure>
  

  More Info:📚

  MDN Figure Element

  If you have any questions or need further clarification, feel free to reach out to me.

  Happy Coding!🎄🎁

  0