Hi Isaque803
Nice job here! Here's a few points you might like to consider:
- You could make the burger-menu element a
button
rather than adiv
. This will ensure that it can be opened by keyboard users. - Perhaps have a look at the
srcset
attribute? This will means you won't need to always load two images to allow for wide/narrow screens, which is much better for performance. - I think you need to put an accessible label on the svg icons at the bottom of the page? You could use
aria-label
(I think they should also be links?).
Otherwise nice job on this one!
Cheers Dave
Marked as helpful
1