Project 8: html, css, flexbox, meadia query

Hi Mainul, nice job! 😄 I have a few suggestions for you:

• This should be a landing page so you have to avoid vertical scrollbar that you have
• Add transition on button and svg images
• Heading should be bigger

On mobile screen is everything okay but in my opinion button is too large and heading is too small

👋Hi Mainul Islam!

I've some feedback on this solution:

• For any decorative images, you should leave the alt="" empty and add aria-hidden="true" to make sure all screen readers ignore those images. In this case, the social media icons and the illustration are decorative images.
• alt value should be informative and human readable, not hyphenated.
• For the logo, just use the company name for the alt="Huddle". You don't need to say "logo".
• Don't limit the height of the body element. Use min-height instead.
• Removed all commented code. Remember, development code is not a production code.

.social-icon{
  text-align: right;
  padding: 3rem 7rem;
  /* margin-bottom: 0; */
  overflow: hidden;
}

That's it! Hopefully this is helpful!

Hello Mainul! Congrats on your submission. Looking very good. Some things to note:

• You need to add background-size: cover and background-repeat: no-repeat to the body to prevent background image from repeating on desktop view.
• Add aria-label to your <a> to resolve the html issues raised in the report.
• You have a small typo error in your .btn text (Registar)