Josh• 320
@Ao-chi
Posted
Hi! You did great doing this challenge!
I saw your navigation menu on mobile version and it looks like it is not on top of the body. To make it appear on top you could add z-index: 1
to your nav-menu class.
Some tips for best practices.
- You should wrap your main contents into
<main>
element for more accessibility. - Add an alt attribute to your
<img>
elements like this<img src="/images/client-databiz.svg" class="clientimg" alt="hero banner">
- You should not wrap a
<button>
inside an<a>
element and vice versa. It is for accessibility practice. You can use either one of them. I you want the button to have a cursor pointer, you can just add this on your css stylingcursor: pointer
Hope this helps! Happy Coding~
Marked as helpful
0