@samuelpalaciosdev
Posted
Hi, Pedro👋
Great job on this challenge👍
I only suggest some things 😉:
-
Maybe, you don't need height on the cards. I mean, setting an explicit height, could cause you some problems, when talking about responsivenes, you could try to get the same result using padding. As I'm seeing on your project, the cards on desktop wiewport lacks content, that's because of the height you set.
-
On mobile view, there's some scrollbar. I'd set an
overflow-x:hiddenon thebody. The content is too big to fit in the specified area on mobile view, setting anoverflow-x:hiddenwould help.
I hope this would help you, have a nice day, keep coding!💙
@samuelpalaciosdev
Posted
Also, there're some HTML issues on your code. I would suggest you whenever you make a website, use W3C Markup Validation Service
@Pedro-Vitor-SG
Posted
@samuelpalaciosdev Wow, thank you so much for the comment, helped a lot. I'll do better!!
