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

mobile first, flex box with alternating direction

P
Haydn Kerr• 535

@haydnkerr


Design comparison


SolutionDesign

Solution retrospective


Code is a bit messy and definitely could use some re writing. Overall I had fun with this one using some custom icons made from illustrator

Community feedback

Vanza Setia• 27,795

@vanzasetia

Posted

👋Hi Haydn Kerr!

I have some feedback on this solution:

  • For the try it free and get started, you should use anchor tag instead of button. In real website, it will navigate the user to a certain page.
  • Try to use HTML semantic tags for wrapping each section, like using section.
  • Don't use br for spacing. You should do it in your CSS.
  • Use heading in sequence starting from h1, h2, h3 and so on. The copyright should be a paragraph not a h6. Don't use h6 to make it look small and bold 😉.
  • The about us, career and so on, should be a link, not just a list item.

That's it! Hopefully this is helpful!

Marked as helpful

1

P
Haydn Kerr• 535

@haydnkerr

Posted

@vanzasetia

thank you for the feedback, I wasn't aware that headings had a strict sequence to follow.

I've definitely got some updating to do

0
Ahmed Faisal• 5,095

@afrussel

Posted

Use outline none for button when click. There are a border showing after click in mobile view

Marked as helpful

1

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