@elaineleung
Posted
Hi Toni, great job in your first attempt at a Junior level project, and I think this looks well put together! 😊
About the questions you asked, I'll try to answer a few here:
-
The ad box width: What I would do is to use
min()
in the width, as this allows the browser to find the best max width depending on the container width. This is what I'll write:// starting mobile view .get-started { width: min(100% - 2rem, 25rem); // 2 rem equals to 1 rem left and right margins combined margin-inline: auto; } @media (min-width: 700px) { .get-started { width: min(70%, 35rem); // You can experiment with these values to see what works } }
-
Centering social icons: What I'd do is to wrap the icons with an
a
tag and turn those them into circles (I also would use<a>
tags wrapped around the spans):// HTML <span class="social-class-circle"> <i></i> </span> // CSS .social-icons-circle { display: grid; place-content: center; border: 1px solid #fff; width: 2.25rem; height: 2.25rem; border-radius: 50%; }
-
Lastly, I notice that on large screen sizes the content tends to stretch out a bit. I suggest adding a
.container
in each of the landmark sections (i.e.,header
,main
,footer
) where you'd add a.container
as a direct child and place all the content of that landmark section inside. That way you can easily adjust and control the width of all the content and get it to be aligned!
Hope this helps you out a bit, and good luck 😊
@ToenSh
Posted
@elaineleung Thanks a lot for taking the time, this was really helpful!