This looks pretty good I think.
A few issues:
- all the main page content needs to go inside main. The about section cannot be outside
- If this was a real site you wouldn't want the h1 inside header. Header is meant for repeating content on every page of a site but h1 is for the individual page title. I think its fine for this because it's just a landing page but worth being aware of for future projects
- Make sure you set a max width for the page content. Zoom right out on a desktop browser and you will see how the design breaks
- An alternative (and arguably better) way to do those curved sections would be with absolutely positioned pseudo elements and svgs or clip path etc.
- "Mobile phone mockup" is not a proper description of that image. If you think it's meaningful and needs a description, then it needs to be a good enough description for someone to be able to picture the image from reading it. If it's decorative, the alt can be blank
- I recommend using an ordered list for the 3 numbered sections
- For the numbers, they should still be in paragraphs. You could even use the css counter to display them maybe.
- "Workit logo" is not good label for a home link. For this I think you need the alt to be
WorkIt - Home
and the href should be"/"
- The social logos definitely need to be anchor links and their alt needs to only be the name of the platform (remove the word 'button')
- ⚠️ NEVER do this:
font-size: 16px;
This one is really important. I wrote about why
Marked as helpful