@jacob-stone9554
Posted
Hi Ram! Overall, this is a great start to the solution!
I do believe that it could be improved upon in a few ways to get it closer to the original design.
In your body styles, you could try background-color: hsl(212, 45%, 89%);
instead of background-color: hsl(220, 15%, 55%);
in order to achieve the background color designated in the design.
Additionally, you could import a Google font to use in your solution. This one recommends "Outfit" which can be found here with instructions on how to embed it on that page as well.
As far as making the design responsive, you have a great start by including a media query. One thing you could try is removing the body
rule from inside that query as it should be applied globally, and really doesn't need to change depending on the size of the viewport. Inside the .container
rule you could try setting the height
and width
attributes to a set percentage like 75% or 80% instead of the full size of the viewport.
One final note, although this is a relatively small solution it would still be good to include semantic HTML. Tags like <main>
, <nav>
, <section>
and so forth.
I hope this helps! Let me know if you have any questions, happy coding :)
Marked as helpful
@Onlyy-Ram
Posted
@jacob-stone9554
heyy Jacob, thanks for the feedback!! it's understandable now.
i'd love to hear from u again buddy :)