@minthing
Posted
container is not centerd (if the display is bigger than 1552) how about use justify-content: center and remove margin:50px in wrapper class?
nice work! cheer! ✨
@Art-wdt
Posted
@minthing Thanks for the great advice!
Looking to hire developers?
Hey! This is my new work. I would be glad to any advice:) Any suggestions you might have! Any comments!
Feedback definitely appreciated
@ApplePieGiraffe
Posted
Hey there, Art! 👋
Well done on another challenge! 👍 Your solution looks good! 😀
I'd like to suggest using a less-important heading tags (such as <h2>
or <h3>
) for the titles of the sections of the card since it's generally considered a best practice to only use a single <h1>
tag per page.
Also, perhaps consider centering things with flexbox (as opposed to margin or padding). In this case, it would make centering the card component in the viewport quite easy, I think! 😉
Keep coding (and happy coding, too)! 😁
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord