@RocTanweer
Posted
Hello π @Ron
Great π work after so much struggle βΊοΈ as you mentioned..!
Suggestions π :
-
Input fields and buttons need a label to be identified on the accessibility API...since we do not wish a label in this case, so we have to use aria-label='a name here' as an attribute in place of them.
-
Try to put some transition when mobile menu is toggled to increase UX.
Hope it helps(happy coding π)
@rontoyhacao
Posted
@RocTanweer Oh right I forgot the aria-label, thanks for pointing out. I know this challenge is on the easy category but I did my best. And to mention, the mobile menu actually has a transition. It has the opacity transition, I just made it subtle. Thank you for the feedback though, I appreciate it.