Design comparison
SolutionDesign
Solution retrospective
Any feedback on better ways to implement CSS Grid would be greatly appreciated
Community feedback
- @kmr-mayumiPosted 10 months ago
Hi @Hunter Mann! I'm a beginner too but I would like to give some observations that I noticed...
- Just for UX reasons, I would recommend using something like
transition: .3s;
in titles that are links; - I noticed that you used the
justify-content: center;
andalign-items:center;
properties, but you didn't use thedisplay: flex;
in the body, which I believe was the goal to center all the content, but Because of this, he ended up using some tags likemargin-top
to try to centralize.
Well, other than that I didn't notice anything much. In fact, as I'm not familiar with using
display: grid;
, with your solution I was able to learn more! Thank you very much :D1 - Just for UX reasons, I would recommend using something like
- @HassanArafa53Posted 10 months ago
Great job on using grid system!, other than @kmr-mayumi's notes, it looks great but it would've been perfect if it was responsive on mobile, i'm guessing you haven't reached media queries yet..until then keep on doing great!
0
Please log in to post a comment
Log in with GitHubJoin 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