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

huddle landing page using flexbox

Ornella T 425

@ornel77


Design comparison


SolutionDesign

Solution retrospective


Hello,

If you have any critics or advice feel free to write them :)

Community feedback

P
AjeaS 585

@AjeaSmith

Posted

Awesome job! like @emmy-html said, your solution is very close to the design. So again, awesome job on that. I noticed that you have a few accessibility issues, which I see are an easy fix. It's basically saying for best practices when working with links <a></a> tags, if the link is not actually going anywhere like to navigate to a different page it can look like this <a>my link</a>. Or, if you are navigating to a different page then, using the href attribute like you did before is the way to go. Nonetheless, great work!

P.S. hope that made sense :)

1

Ornella T 425

@ornel77

Posted

@AjeaSmith oh ok yeah that make sense thank you so much :)

1
P
AjeaS 585

@AjeaSmith

Posted

@ornel77 your very welcome!

0
emmy-html 345

@emmy-html

Posted

Hi! I think it looks great! The responsiveness is really good and it doesn't look like you missed anything when it comes to the design vs your solution. One suggestion I have is about the smallest size available. In the footer, all of the content is left aligned. I think if the "Huddle" logo and the Facebook, Instagram and Twitter content was centered, it would look better aesthetically. Great job!

1

Ornella T 425

@ornel77

Posted

@emmy-html ah ok thank you very much :)

1
emmy-html 345

@emmy-html

Posted

@ornel77 no problem!

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