flexbox responsive

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted about 4 years ago

  Hey, Javier! 👋

  Good work! 👍

  In addition to brasspetal's great feedback,

  • Adding a hover state to the button and social media links would be a nice touch.

  Keep coding (and happy coding, too)! 😁

  0
• 

  Anna Leigh•5,135

  @brasspetalsPosted about 4 years ago

  Hi, Javier! This responds nicely, and could be even better with just a few small changes:

  • To clear up the accessibility errors, add "#" to your hrefs: <a href="#">
  • Adding some padding or margin to your container will help it more closely match the design.
  • Adding cursor: pointer to your button will better show that it's clickable.
  • Decreasing the font sizes on the mobile layout.
  • Changing to the mobile background for the mobile layout.

  0