@mattstuddert
Posted
Hey Saurav, what mistakes do you feel that you've made while building this project? Here are a few pointers after taking a look at your code:
- You're not loading in the
700
weight that you need of Montserrat to display the content correctly. This is making your content appear thinner vs the design. - You're using multiple
h1
elements on the page. I'd always recommend only using a singleh1
, which is your main page heading. - You've got
h1
,h3
andh4
elements, but noth2
. Avoid skipping heading levels unnecessarily as it can lead to accessibility issues with the content hierarchy.
I hope these tips help. Keep up the great work!
@sauravchamoli17
Posted
@mattstuddert I've fixed the multiple h1 present in the project but I am not able to fix the font weight of the content and the background pattern is also not visible. Can you help me out with this?
@mattstuddert
Posted
@sauravchamoli17 to load in font-weight: 700
from Google Fonts you'll this link
tag instead of the one you've got: <link href="https://fonts.googleapis.com/css?family=Montserrat:700&display=swap" rel="stylesheet">
. I'd recommend having a deeper look at Google Fonts to see how you can customise the fonts that you load in to add different weights.
For the background patterns, you've just got the wrong path for the images. You need to change url(/images/bg-top.svg), url(/images/bg-bottom.svg)
to url(../images/bg-top.svg), url(../images/bg-bottom.svg)
.
@sauravchamoli17
Posted
@mattstuddert Thanks.