@mattstuddert
Posted
Hey Khaled, nice work. The main thing I'd recommend working on with this project would be to make it accessible. At the moment, you're using div
elements instead of buttons for the keys. div
elements aren't interactive and so can't be accessed by people not using a mouse/trackpad to click on the keys. Always be sure to use interactive elements if you expect the user to interact with the page.
Also, you're not using any of the technologies you've added as tags. I'd recommend removing the tags so that they match up with what you've used to build the project.
Keep it up!
Marked as helpful
@TunUnpred
Posted
@mattstuddert Thanks for your time Matt, all buttons are now interactive and there are no unused tags.
@mattstuddert
Posted
@TunUnpred, you're welcome. Nice work! That's functioning much better now 👍