LetsFrontend• 500
@B1N4R1
Posted
Hey Roberto!
Firstly congrats on completing the challenge! Your solution looks good!
Here some pointers:
- You should really get rid of the
min-width: 1440px
in the 1440px media-query. This is just streching the card way to much - Try using lesser height definitions like you do in the
div.main
element, you can get rid of that height. Also when creating buttons, don't use like aheight: 40px
it's better if you use padding like apadding: 15px
- Try using more semantic html and not so much divs, here an article about it.
There is always room for more improvements but for me this are the more pressing ones in your case
Hope this helps,
Cheers!
PS.: You should also look at the report and try to fix the errors it tells you
Marked as helpful
1