HTML, CSS and Flex

Community feedback

• 

  Muhammad Shajjar•1,100

  @muhammadshajjarPosted about 3 years ago

  Hey, awesome work on this one 👍 The layout looks good and responds well on all viewports but it is not perfectly centre on viewports greater than 1440px, you would see the left alignment of hole content margin: 0 auto on body would fix it.🙌

  Some more suggestions that would make your solution more cleaner are

  • I prefer to put a header outside of main element
  • Avoid adding those words that relate to graphics such as logo, image, and for decorative images use alt attribute as a Blank atl=“” and use aria-hidden="true" or role="presentation". In your case .service-images are decorative
  • Hover states on Social media images are missing, Try to add them too.

  Marked as helpful

  0
• 

  Carla•420

  @iamcgsPosted about 3 years ago

  Thanks for your feedback. Your comments are really helpful :)

  0