HTML, CSS (Grid) ;-

Community feedback

• 

  T

  Matt Studdert•13,611

  @mattstuddertPosted almost 5 years ago

  Hey Shahmir, nice work on this challenge! I've only got a couple of small recommendations:

  • I'd recommend using a ul instead of a p with br tags. This is because the content is a list of features/benefits, so a ul makes sense.
  • On a future project, I'd recommend having a go at using min-width media queries instead of max-width. It often leads to less code and also has the benefit of loading in fewer styles for mobile users.

  Keep up the great work!

  0

  ---

  

  Shahmir Faisal•480

  @shahmirfaisalPosted almost 5 years ago

  @mattstuddert Thanks for your suggestions I just realized that, I forgot ul tag and using a p tag at that point doesn't makes any sense. Thanks ;)

  0