CSS Grid and Flexbox

Solution retrospective
Some problems with Vercel but live site is working.
Hello again! I worked on this challenge quite while. Publishing it before i peak others code. I'm sure its overcoded and contains many mistakes. If You have a spare minute, please check the code and write me some suggestions. I'm looking forward for constructive criticism, and i'm really motivated to learn more :).
Cheers for everyone !
Please log in to post a comment
Log in with GitHubCommunity feedback
- @ApplePieGiraffe
Hi, n0macx! 👋
Nice work on this challenge! 🙌 Your solution looks great and is responsive! 👍
I suggest,
- Adding a
max-width
to the main container or wrapper of the page to prevent the content of the page from becoming too wide on extra-large screens. - Adding a
max-width
to the testimonial cards to prevent them from looking too stretched when the layout first changes from desktop to mobile. - Centering the review boxes in the mobile layout of the site.
- Taking a look at your solution report and trying to clear up some of the errors that are there (some of the HTML issues might be because of Vercel, maybe—but there are few a issues towards the bottom about nesting HTML and writing semantic markup that you might want to take a look at). 😉
Keep coding (and happy coding, too)! 😁
- Adding a
- @n0macx
Got technical problems with live site :)
Join our Discord community
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