Design comparison
Solution retrospective
I am particularly proud of having managed to efficiently implement a responsive design with CSS Grid and Flexbox, while integrating variable fonts. The layout adapts perfectly to different screen formats and the use of CSS Counter
for the recipe steps was a real plus to make the content dynamic and easy to maintain.
If I had to do this project again, I would spend more time on accessibility.
What challenges did you encounter, and how did you overcome them?One of the biggest challenges was managing the CSS grid for the nutritional information. I wanted each row to be well aligned with the labels on the left and the values on the right, while keeping a consistent underline. By using CSS Grid and adjusting the alignment with grid-template-columns
and nth-last-child
, I managed to achieve the desired rendering while keeping a flexible and maintainable structure.
I would like to get more specific feedback on using CSS Grid in my project, especially for the layout of the nutritional information. While the current layout works well, I am wondering if there are more optimized techniques or best practices I could apply to make the grid even more efficient, flexible and maintainable. Also, I would like some advice on managing bullet points in lists.
Community feedback
- @nathan-perkinsPosted about 1 month ago
Hello @laurentGurbala, congratulations on completing this project! I really enjoyed seeing the way that you organized your CSS into various files; I may play around with that in future projects.
I wish that I could give you feedback on using grid for the nutritional information, but I do not feel that I know enough about CSS grid at this point to give confident advice in that area.
One thing I did notice as I was reviewing your HTML was that you employed section elements without using an aria-labelledby or aria-label attribute. I learned as I was working on this project that the section element does not automatically include the role of "region", you must add an aria-labelledby, aria-label, or title attribute to incorporate it as a semantic element. The way you currently have it set up, it essentially acts just as a div.
I hope this helps, good luck on future projects!
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