Huddle landing page with single introductory section using CSS Grid

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted almost 2 years ago

  Hi!

  Regarding the social media icons, I prefer the original design. Using the black color for the social media icons while all the other text is white is not appropriate. That's my opinion. I am not a designer. So, feel free to take my suggestion if you want.

  Some suggestions:

  • Why are you adding aria-hidden="true" to the img that has alternative text?
  • Logo is an important content of the page. It's not decorative. So, it should not have aria-hidden="true" attribute.
  • Also, the alternative text for the logo should be the company name (alt="Huddle"). Currently, people can think that the site is a website for a Logo company.
  • Never limit the height of the body element. It will not allow the users to scroll the page if the page content needs more height. You can see the issue by looking at the site on a mobile landscape view. So, my recommendation is to use min-height instead.
  • Use the em unit for media queries. It adapts when the users change their font size setting. Here are some references.
    • @media - CSS: Cascading Style Sheets | MDN #accessibility_concerns
    • PX, EM or REM Media Queries?

  I hope this helps!

  Marked as helpful

  1

  ---

  

  NT-squared•110

  @nt-squaredPosted almost 2 years ago

  Hi 👋 Your comment is always helpful, especially regarding accessibility. I feel very grateful for that. Thanks a lot!

  0

  

  Vanza Setia•27,775

  @vanzasetiaPosted almost 2 years ago

  @nt-squared Sure! 👍

  0