This looks really great, well done! Very nice solution
The only things I'd tweak are in the footer, all relating to semantics
- I'd make phone and email into links
- I'd wrap the nav items in the footer in a nav element (labeled with aria-label footer)
- I'd make that nav list into one list and use the column property on larger screens
- if wrap those social icon in anchor tags, label them, set them to open in a new window and use
rel="no opener"
Then after all that I'd make sure interactive elements all have clear and consistent focus-visible states.
Pretty small suggestions really, as this is already great ☺
@LesleyWesley
Posted
@grace-snow Thanks for the suggestions! :) I'll try to make those changes when I get a chance!