Hello, Andrew! 👋
Good effort on this challenge! 👍
Regarding the background image, on the desktop layout, I set the
background-position: center right and I think it looks similar to the design.
Some suggestions to improve this solution.
- I would recommend using the
logo.svginstead of creating the logo by yourself. With the existing logo, all you have to is to put the logo using
imgtag and add an alternative text like Huddle.
- Alternative text for images should not contain any words that related to image (e.g. picture, photo, logo, icon, graphic, avatar, etc). It's already an image element so the screen reader will pronounce it as an image.
- Not all images need alternative text. In this case, the illustration doesn't need alternative text so I would recommend leaving the
alt=""so that screen reader users won't have to listen to unrelated content.
- Each social media icon should be wrapped by link element and I would recommend removing the
<ion-icon name="logo-facebook"></ion-icon>tag since the site is not using Ionicons.
I hope you find this information beneficial. Happy coding! 😄
Marked as helpful