SCSS, javascript, flexbox

Community feedback

• 

  T

  Matt Studdert•13,611

  @mattstuddertPosted almost 5 years ago

  Nice work on this, Ben! Your solution looks really good. I've got a couple of small suggestions, but nothing major:

  • I'd use a form element instead of a div with a "form" class for the email input. You can then listen out for the submit event on the form, as opposed to a click on the button.
  • I'd also recommend adding anchor tags around the social icons and making them accessible. As you'd expect these to be clickable links if this were a real landing page.

  Keep up the great work! 👍

  0