- on line 38 in your html file there is a navigation list which is duplicated again from line 65. The reason I see here was to have a navigation for mobile and another one for desktop but with responsive design such cases should be avoided. The navigation can be placed properly using css.
- the hamburger div is supposed to be a button, a div will require a lot more coding for it to be accessible to assistive technology users. Just change the div to be a button and leave the span elements as is. Provide content that can be read a screen reader so that assistive technology users will be able to navigate your site without confusion. In fact they should be a hamburger image in the assets folder. Find about css class Sr-only, so that you can visually hide content but the same content should be read by screen readers
Marked as helpful
@UDsGitHub
Posted
@ChamuMutezva Ah thanks for clearing that up. I really appreciate your feedback and I would work on those changes right away.