Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

fully responsive huddle landing page pure css

Prem mali 240

@prem1835


Design comparison


SolutionDesign

Community feedback

P
Matt Studdert 13,611

@mattstuddert

Posted

Hi Prem, nice work on this challenge your solution looks really good! Here are a few recommendations after taking a look at your code:

  • You've currently got 5 h1 elements, but no other headings. I'd recommend having just a single h1 to mark the page title and then use other heading levels to create a content hierarchy.
  • Try to use the alt text in the img tag as a description of the image. Your alt for the logo is just "logo" and the main illustration is "header image", which adds no context to what the image is. In these instances, alt text like "Huddle" and "Illustrative mockups of desktop and mobile screens" would add more context.
  • The social icons would be clickable links if this were a real landing page, so I'd put anchor tags around each icon.

You've done a really good job on this. Keep up the great work! 👍

1
Prem mali 240

@prem1835

Posted

Thank you Matt for your suggestions I will work on the all of them. I really appreciate your feedback ♥️ Thank you

0

P
Matt Studdert 13,611

@mattstuddert

Posted

@prem1835 you're welcome Prem!

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord