Design comparison
Solution retrospective
While refactoring my markup and styles for this project, I was reading the Tailwind docs and found out that the font size utility class can take a modifier for line height. Previously I was leveraging font size and line height like so: text-sm leading-normal sm:text-lg sm:leading-normal
. However with the modifier, there isn't a need for the additional utility class, which I think cleans up the code nicely: text-sm/normal sm:text-ls/normal
.
I did get a little too focused on the Figma design file on this project. Specifically when it came to the border on the card. In my first iteration of the markup and styles, I used the ring
utility class to create the border as the Figma file showed in "inside" border style for the card. However, I neglected to realize that because of the default box-sizing: border-box
on each element provided by the reset of Tailwind, borders are inherently inside the elements. I did fix this by returning to the normal border
utility class after refactoring my code.
Community feedback
- @hadil-rjbPosted 4 months ago
Good job, but there's a mistake in the word "structure" and "Foundations" — the "F" should be capitalized.
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