Design comparison
Solution retrospective
Becoming familiar with Git, GitHub, and GitHub Pages. Looking forward to the next challenge.
What challenges did you encounter, and how did you overcome them?Making responsive text based on image size with only CSS. Just set the following properties on the text element:
- width to 0: so that the text will not contribute in the sizing of the container.
- min-width to 100%: So that the text will be as wide as the sibling image.
What is the best way to make this responsive and compatible with old browsers.
Community feedback
- @MikDra1Posted about 2 months ago
If you want to make your card responsive with ease you can use this technique:
.card { width: 90%; max-width: 37.5rem; }
On the smaller screens card will be 90% of the parent (here body), but as soon as the card will be 37.5rem (600px) it will lock with this size.
Also to put the card in the center I advise you to use this code snippet:
.container { display: grid; place-items: center; }
Hope you found this comment helpful 💗💗💗
Good job and keep going 😁😊😉
Marked as helpful0@BarbarossaNextPosted about 2 months ago@MikDra1 Thanks for the great tips 💗 There is something else I'd like to ask about I read that I should use relative units such as em and rem, instead of px. However, Figma has no notion for these relative units. Should I do the conversion calculations from px to em and rem manually? What are the best practices to do that?
0@MikDra1Posted about 2 months ago@BarbarossaNext As you start you can use a converter but after some time you will realise that you will do it without much thinking
Marked as helpful0 - @wagnermoschiniPosted about 2 months ago
This solution is a bit different visually
The card must be 320px width and 499px height, and in your solution was 320px width and 533px height. It's because you put an extra space between image and content that not was as the figma spec that was the space-300 property from design system file.
Also the title right color is slate-900 (#1f314f) and the text is slate-500 (#68778d)
Marked as helpful0@BarbarossaNextPosted about 2 months ago@wagnermoschini Really appreciate your helpful review. I updated the project accordingly and still have a couple of questions:
-
The overall card height is now 499.78 px instead of sharp 499 px (as in the Figma design file). It turned out that the title text is the element with this additional 0.78 px. I could not recognize the source of this error although I tried my best copying all font-related properties as in the design file.
-
The comparison between the design and my solution still shows a significant difference in height although Dev Tools show only 0.78 px difference.
-
I read that I should use relative units such as em and rem, instead of px. However, Figma has no notion for these relative units. Should I do the conversion calculations from px to em and rem manually? What are the best practices to do that?
0@wagnermoschiniPosted about 1 month ago@BarbarossaNext
For this challenge specifically, maybe we don't need the card height flexible according to the content, so it's not a problem if you put height:499px in your .card class IMO.
EM and REM are proportional units and both needs a parent size reference, the diference between them is that EM looks to the direct parent font-size reference and REM looks to the root font-size reference that is the font-size value of the html element. This approach can simplify layout scale between responsive breakpoints at more complex situations than that. IMO I don't think this approach can do a relevant improvement for this challenge maybe for some next more complex challenges in the future that will challenge you to do huge changes in layout/font scale between devices.
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