Design comparison
Solution retrospective
Should have started with flexbox
What challenges did you encounter, and how did you overcome them?using flex
What specific areas of your project would you like help with?none
Community feedback
- @carstenkoernerPosted 8 months ago
Hi, I have also just uploaded my first solution. But I would still like to make a few suggestions.
-
Firstly, read Grace’s comments on my solution. You can use them completely for your solution. Link to Grace’s comments
-
You can use rem instead of working with px everywhere. The advantage of using rem is that all sizes depend on an initial size. If you adjust this - all other sizes and ratios adapt. This makes it easier to adjust responsive displays. The default font size in this project should be 15px. You can use this as the font size for the body-element and thus serves as the initial size, as it is inherited by all subordinate elements. By setting the initial size, you define 15px as 1 rem. So if you want to use 20px somewhere, for example, you convert this to 1.6rem.
-
Read the definition of BEM. BEM can help you to bring a better structure to your CSS. This is not necessary in small projects like this one, but can help you later in larger projects. https://getbem.com
-
Your page does not behave responsively if the screen width becomes smaller. In this project, it is sufficient to work with max-width and not define the height at all. Height must be able to adapt flexibly via the content.
JM2C
Marked as helpful0@princeezePosted 8 months agoHi, @Carsten. Thanks a lot for the detailed solutions and helpful resources. I have implemented all of them. Let me know if you have any other thoughts.
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