Design comparison
Solution retrospective
👍 Any suggestions on how I can improve are welcome!
Community feedback
- @RagudosPosted 7 months ago
Great job! I like that you specify parts of the CSS code, which is great for large files.
I recommend trying out flex, align-self, and height in contrast to your grid-row, grid-column solution.
Also, I noticed that you are very specific with your class names, which is great, and it may be better to use generic class names for stuff like images (e.g. .profile-circle__lg, .profile-circle__xl).
On a side note, I like your GitHub Readme. Keep it up, man!
Marked as helpful1@alberto-rjPosted 7 months ago@Ragudos, thank you very much for your suggestions and compliments!
I didn't use Flexbox, because I'm trying to understand CSS Grid. But I will definitely improve my solution soon.
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