- use semantic elements where possible, using a div in the place of an interactive element(in this case a button) is not beneficial to assistive tech users.
- the following elements has been duplicated (some for mobile and some for desktop) - in my opinion that is not best practice in terms of responsive design. View the structure of your html without your css and try to check on the flow.
<ul>
<li><a href="#">About</a></li>
<li><a href="#">Services</a></li>
<li><a href="#">Projects</a></li>
<li><a href="#" class="nav-btn">Contact</a></li>
</ul>
- look into responsive images techniques (picture element, srcset) when you want to switch images between mobile and desktop , avoid the following kind of display:
<img
src="./images/mobile/image-transform.jpg"
alt="transform"
class="mobile"
/>
<img
src="./images/desktop/image-transform.jpg"
alt="stand-out"
class="desktop"
/>
- alt values have to be descriptive to benefit assistive tech users (they should be able to visualize the message that is being put across)
0