Grid and flexbox

Community feedback

• 

  hardy•3,660

  @hardy333Posted about 3 years ago

  Hey, website looks very good and clean. It is also responsive - good job.

  Several suggestions here:

  • You can use <main></main> semantic element instead of role="main" on div element, using HTML 5 semantic tags are considered better practice, you can also combine both of them.
  • try to use <ul></ul> instead of <p> and <br> tags on last section, those are list and list items more then hole paragraph...

  Good luck and have a nice day.

  Marked as helpful

  3
• 

  Benjo Quilario•1,810

  @benjoquilarioPosted about 3 years ago

  Great job on finishing this one, It looks good and response very well.

  Aside from hardy suggestion, My small suggestion is to add a cursor: pointer; sign up button, to inform the user that this is clickable.

  keep coding!

  Marked as helpful

  2
• 

  HappyAtom•55

  @HappyAtom3Posted about 3 years ago

  great job, everything looks very good.

  Congratulations on solving this challenge.

  1