Skip to content
Submitted almost 4 years ago

CSS flexbox

Tunde Xzabhn•50
@Awesauban
A solution to the Huddle landing page with alternating feature blocks challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

I would like to get feedbacks on better to structure a lot of things here better. Please do check it out and drop your fedbacks. Thank you

Code

Please log in to post a comment

Log in

Community feedback

  • Tunde Xzabhn•50
    @Awesauban

    👋Hi Vanza Setia

    Hope you're doing great?!

    I have made corrections to some of your feedback. Please do check it out and let me know if i have to still make some more corrections.

    Thank you!

  • Vanza Setia•27,715
    @vanzasetia

    👋Hi Tunde Xzabhn!

    I have some feedback on this solution:

    • It would be better if you use section instead of div for every section.
    • For any decorative and meaningless images, you should leave the alt="" empty. That way the screen reader can just ignore those images.
    • For the About Us, What we do, and so on, you should wrap them in anchor tags. In real website they will redirect the user to another page.
    • The hero section should not be inside the header. The header as banner only contains navigation links and sometimes company logo.

    That's it! Hopefully this is helpful!

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