Huddle landing page using SASS

Community feedback

• 

  バレンタイン 😈•64,190

  @VCaramesPosted almost 2 years ago

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

  • Remove the Article element inside the Header, it is unnecessary.

  • Your "buttons" were created with the incorrect element. When the user clicks on the button they should directed to a different part of you site. The Anchor Tag will achieve this.

  • Your "buttons" need to have a cursor: pointer, hove, and focus-visible.

  • 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.

  • The "Ready To Build Your Community?" should be an H3 heading.

  • The "bottom-links" need to wrapped in a nav element and each one should ne wrapped in an Anchor Tag.

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

  Happy Coding! 🍂🦃

  Marked as helpful

  1

  ---

  

  Allesson Figueiredo•110

  @allfigueiredodevPosted almost 2 years ago

  @vcarames

  Thank you! Helped a lot!

  Already did the changes.

  0