SCSS, grid, flexbox layout and JS

Hi, Ali! 👋

Nice work on this challenge! 👍 It's great that you've created a hamburger menu for small screen sizes which makes it looks cleaner than the original design in my opinion. However, I would recommend using button element for the button of the hamburger menu. Currently, it's not accessible by keyboard and screen readers because of the div element.

Other feedback.

• Wrap the email input and the submit button with form element. They should live inside the form element.
• If the image has aria-hidden="true" attribute, it's recommended to leave the alt empty. The purpose of aria-hidden="true" is to hide the image from the VoiceOver screen reader because it's a bug where it still pronounces the empty alt images when it shouldn't. So, if it's a decorative images then leave the alt empty and add aria-hidden="true" to hide it from the VoiceOver screen reader.
• Good job on using aria-label on the social media links! 👏
• Always use classes to reference all the elements that you want to style. Using id is going to make your stylesheet have high specificity (hard to maintain).

I hope this helps! Keep up the good work! 👍

P.S. I would recommend removing the #solid-js tag because the site is not built with SolidJS