@mattstuddert
Posted
Hey Muhammad, nice work on this challenge. Your code doesn't look messy at all. It looks like you've got a good code architecture and a system for doing things, which is great. The main piece of feedback I'd have is to avoid setting click listeners on non-interactive elements. For example, you've got click listeners on the li elements for the tabs. The li element isn't interactive by default, so it can't be focused for selection by people who cannot use a mouse/trackpad to click. This makes it inaccessible, which isn't good. Instead, you could use a button or a element and set a click listener on that. This means people using mouse/trackpad or keyboard and other assistive technologies can select the element to change the content.
I hope this helps. Your solution looks really good. Keep it up!
@fadzrinmadu
Posted
@mattstuddert i will remember it well thank you very much for the comment, this is very helpful. I will try to learn more about it
@mattstuddert
Posted
@fadzrinmadu you’re welcome! Smashing Magazine articles often talk a lot about accessibility, so I’d recommend looking out for them.
@fadzrinmadu
Posted
@mattstuddert okay i will check it
