Html Sass flexbox grid

Community feedback

• 

  T

  Matt Studdert•13,611

  @mattstuddertPosted about 5 years ago

  Awesome work on this challenge, Pedro! 👍

  Here are a few pointers after taking a look at your code:

  • You currently don't have a h1 on the page, which I'd always recommend having.
  • Your input should have an aria-label attribute, as it isn't paired with a label. This will make it accessible to screen reader users.
  • I'd recommend removing the br tags from the paragraph, as you'll see on mobile it makes the text break in awkward places. Instead, let the width of the containing element dictate the width of the text.

  Keep up the great work!

  0

  ---

  

  Pedro Germano•140

  @pedroGermanoPosted about 5 years ago

  @mattstuddert thanks for the advice =)

  0