Design comparison
Solution retrospective
Hi! This is my first junior-level challenge and It really helped me to practice my skills with CSS Grid. I would appreciate any feedback! :)
Community feedback
- @yvsminvsPosted over 3 years ago
Hi @AgataLiberska!
Thank you!! I named one of the classes as ''head'' as a random name but I understand that it can be a little bit confusing... I'm going to modify all these things that you've mentioned!
And once again, thank you so much for the tips!! :)
0 - @AgataLiberskaPosted over 3 years ago
Hi @yvsminvs, well done on this challenge, looks great! The only thing I would do is have it form a grid a bit earlier or limit the width of boxes when they are in a column - only because long lines of text are difficult to follow on the screen. Also I'm not sure about using heading tags for names and roles here, not sure if they can be considered a heading in this context? Just something to think about :)
Hope this helps :)
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