@FluffyKas
Posted
Hey, your solution is a good start but there's a few adjustments you could make.
-
Your font-family doesn't work. You need to follow that link that was given to you in the style guide and select the appropriate font-weights of the font-family. Then google fonts will give you a piece of code that you can put in your html or css. Using that link directly in the head won't work unfortunately ^_^
-
You could give your background a
background-size: contain
to cover the whole screen. If you add a background-color as well, it will look like the design. Alternatively, you can use thebackground
property and add both the color and the image, comma separated. -
Take a look at the accessibility report! Semantic markup is very important and the report helps a lot if you're not sure what you're doing. Your alt attribute is also incorrect, if you're not sure what they do, here's this cool article, that will help you: (https://axesslab.com/alt-texts/)
And finally, I'd say as a newbie your focus shouldn't be on reducing your code. Don't stress too much over it, reducing code can come later when you know more or less what you're doing. Then you'll find ways to reduce it yourself and see when it's even appropriate to reduce it. Good luck! :)
Marked as helpful
@Suraj9505
Posted
@FluffyKas i thought the font family worked out but i will check it out and thanks for the feedback it was my first challenge so i didnt knew much about them so thanks for the help i will try to improve it. :)