Design comparison
Solution retrospective
Hey,
This is my first challenge and after 2 days, it feels great to be done with it. I struggle a bit, but I'm satisfied how it end up.
I just want to know, do I have a lot of unnecessary code, because sometimes I just type something, it works, but I don't understand how.
Community feedback
- @zineb-BouPosted almost 3 years ago
Hi Stefan,
Here are some suggestions for your solution:
-
Instead of using
<div />
for the image you can use<picture />
element this will allow you to set different versions of the image(desktop/mobile) -
I noticed you included some CSS resting, check this modern CSS reset.
-
Adding a breakpoints ( min-width:1200px) I think it's a bit early for small screens, try to set it smaller (1200px meant to be for desktop devices)
-
Use rem or % to set the
<.card/>
width you will see it shrinking when transitioning between different screens.
1 -
- @st3wnPosted almost 3 years ago
Hi man,
just a little tip: You can give the left side of the container a padding-right: px; to get the text smaller. Or you put the h1 and paragraph in a div container and give the div container a width.
Aswell you can check the width and height of the image by rightclicking it, click on the tab "details" and then see the px size of the image. Is the image for example 446px high, you set the container to a height of 446px. In your case the picture is a bit cut off.
But overall pretty good!
1 - @st3wnPosted almost 3 years ago
Yo Zineb,
love it, thanks :)
Can you still send me your CSS rest? Would be great!
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