@emestabillo
Posted
Hey @stephjoseph, looks awesome! The only two things that I noticed is that
- There's 2 separate navs for mobile and desktop, which, I think can be accomplished by media queries instead, and
- The 3 tabs on the side should be
button
s not divs, since they are interactive components
Hope this helps!
Marked as helpful
@stephmunez
Posted
Hi @emestabillo, thank you!
Those points help a lot. I decided to separate both navs for decluttering purposes but definitely should try putting both navs into one component and use media queries instead.
You're right on the second point, thanks for pointing it out. Should hopefully pass a11y standards after haha
Have a nice day