Insure Landing Page - SCSS, Grid, Flexbox, JS

@mattstuddert Hi, Matt!

I really appreciate you taking the time to look over the code! I knew having two separate navigations was likely…not the best solution 😅 - thanks for calling me out on it. I’ve gone back in and combined the navigations, and also changed the alt attributes as you suggested.

Thank you for sharing the article on menu accessibility! I’m always uncertain on best practice when it comes to accessibility (and, honestly, just accessibility practices in general), and this is really helpful. I haven’t implemented the aria attributes yet, as I’ll need to give the article a few more reads, but I’m excited to start adding proper accessibility into my projects.

Thanks again for your suggestions! 😄

@brasspetals you're welcome! Accessibility can be tricky, but it's well worth learning about as it's such a critical piece to being a good front-end developer. The Inclusive Components site is an incredible resource for that, so you'll get a lot out of reading that 🙂

@ApplePieGiraffe Hey, APG 👋 - it's definitely been a minute!

Thanks for the welcome back. Hopefully today I'll have time to really get back into it and start providing feedback again. Hope you're well!

Yeah, nothing crazy with this one (that's your department 😄 - I haven't commented yet, but I saw that entire extra Xmas theme on the to-do list challenge. Only you!). Thanks for the feedback again as always! I'm forever paranoid there'll be some major issue I completely blanked on.

Happy coding to you too!

@brasspetals

Haha, I feel you! I'm often scared I'll spend so much time trying to add some cool feature but then I'll totally forget something else that's basic but very important! 😅 (Yet it seems to me that when I'm most worried about something going wrong, not much does and when I'm totally not thinking about it, a bunch of things go awry). Anyway! 😂

👍