mobile first, flex box with alternating direction

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 3 years ago

  👋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

  ---

  

  Haydn Kerr•535

  @haydnkerrPosted over 3 years ago

  @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

  @afrusselPosted over 3 years ago

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

  Marked as helpful

  1