Design comparison
Solution retrospective
I learned how to switch between a column and row layout.
What challenges did you encounter, and how did you overcome them?The quality of the Figma mockups weren't as professional as in other challenges unfortunately. That's why it was a bit hard for me to understand everything correctly.
Community feedback
- @JomagenePosted 3 months ago
You're doing an amazing job with this project! The folder structure is impressively organized, and your use of variables shows great skill—well done! I genuinely enjoyed reviewing the presentation; it’s nearly pixel-perfect in matching the design. The only thing I noticed was a tiny mismatch in some padding, but it's hardly noticeable.
One suggestion to make your project even better is to use
rem
for font sizes instead of pixels. This will help ensure that your design adapts beautifully when users adjust their preferred font size in the browser. You can check out this article for more insights.For accessibility, using the
main
tag as your primary container and wrapping images in afigure
tag would be a great enhancement.I’ve really tried to find areas for improvement, but honestly, you’ve done a fantastic job! Keep up the excellent work—your attention to detail is clear.
One final tip: consider using
em
for paddings instead ofrem
. Since padding is related to the parent element,em
can be a better choice. This resource might be helpful.Keep pushing forward !
1@MichaHuhnPosted 3 months ago@Jomagene Thanks for your feedback!
- I'm actually using
rem
for font sizes. - That's a good idea to use
em
for certain paddings. However, the Frontend Mentor challenges usually use a design system for spacing, e.g.--spacing-200: 1rem;
. That's why I used these spacing variables which are based onrem
. How would you deal with that?
0@JomagenePosted 3 months ago@MichaHuhn You're welcome! I'm glad my feedback was helpful.
Sorry for confusing, I saw letter-spacing:5px, which could be better using em. Great to hear that you're already using rem for font sizes—awesome work! Regarding your question about spacing, using a design system with spacing variables based on rem is a solid approach, especially for consistency across the project. If you want to explore using em for specific paddings while maintaining the design system, one idea could be to create additional variables for those cases. For example, you could define padding variables like --padding-em-200: 1em; that you use when you want the padding to scale with the parent element instead of the root.
That said, if the design system is working well for your project, sticking with rem might be perfectly fine. It's all about finding the right balance for the specific needs of the project.
Keep up the great work, and feel free to reach out if you have any other questions or ideas to discuss!
Marked as helpful1 - I'm actually using
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