@FluffyKas
Posted
Hey,
Please don't tag tools that you don't use in your solution, it's very misleading...
Your solution overall seems fine, but there's a few things you could do:
-
You're overusing <section>. The section named main could be swapped for <main>. All the rest should be just divs (perhaps an <article> for "main-one" if you wanted).
-
Your font-family seems a bit all over the place. I see 4 different font-families right now. This project is supposed to have only one, if I remember correctly. It's the best to declare the font-family on the <body>, that way it doesn't get messy.
-
For the background-image, you should leave the alt text empty. For the profile image, it could be the name of the person, so "Victor Crest". Alt-texts are helpful for people who can't see the images, they're meant to provide a text alternative (this explains why you wouldn't want "background-image" as an alt text announced >.<).
I hope this helped a bit. Good luck ^^
Marked as helpful
@afaiz-space
Posted
@FluffyKas please tell me, what right structure to write HTML code and CSS.
