P
Matt Studdert• 13,611
@mattstuddert
Posted
Hey Shahmir, nice work on this challenge! I've only got a couple of small recommendations:
- I'd recommend using a
ul
instead of ap
withbr
tags. This is because the content is a list of features/benefits, so aul
makes sense. - On a future project, I'd recommend having a go at using
min-width
media queries instead ofmax-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
@shahmirfaisal
Posted
@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