
Design comparison
Solution retrospective
What I’m Most Proud Of:
-
Learning Sass from scratch before taking on this project and effectively applying it to improve maintainability with variables, nesting, and reusable styles.
-
Successfully implementing the cross-like grid layout without using grid-template-areas. Instead, I carefully positioned the cards using CSS Grid while ensuring responsiveness.
-
Sticking to a clean and structured codebase, making the styles easy to manage and read.
What I Would Do Differently Next Time:
-
Experiment with CSS Grid more deeply to see if grid-template-areas could have made the layout even more intuitive.
-
Work on a more refined mobile-first approach to ensure smoother scaling from small to large screens.
-
Optimize the texts styling, particularly spacing and typography, to ensure better readability on all screen sizes.
Challenges & How I Overcame Them.
-
Learning Sass; I had never used Sass before, so I had to learn its syntax and best practices.
-
Positioning the Cards in a Cross-Like Grid Layout; The design required the middle cards (Team Builder & Karma) to be aligned, while the Supervisor and Calculator cards slightly overlapped them. I used CSS Grid with manual positioning, ensuring the layout was responsive without breaking.
I barely get any feedback on my projects, so I would really appreciate any constructive criticism. It would help me understand where I’m lacking and how I can improve. Specifically, I’d love feedback on:
Code Structure & Best Practices: Are there any areas where my code could be cleaner or more efficient?
Accessibility & Readability: Is my design accessible? Are there any improvements I could make for better readability and usability?
Any feedback, big or small, would be greatly appreciated! 😊
Community feedback
- P@GentleHorsePosted 8 days ago
Looks great and I can tell that you deelpy understand the CSS Grid! The result looks exactly the same as the design file provided. One thing I noticed is that the card content is a bit out of screen for middle sized screens such as tablets since the card width is too wide to fit with 3 cards in one row for these size of screens. Maybe it's nice practice to define the media query for tablet or change the breakpoint size? Anyway, in general, great job!!
Marked as helpful0
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