@elaineleung
Posted
Hi Trung, I think this looks quite well put together! If there's one thing I'd change, I would use a class to change the style of the selected button instead of using the :focus
pseudo class, seeing that focus is meant to used for showing which element the user had last interacted with. If you accidentally click on something else like the background or the text, the selected button would go back to its unselected color because it has lost the focus, and that would make it seem like no button got selected, even though in the background JS is still keeping track which button got selected. I would just create a new class called selected
, style the selected button with that class, and then use JS to add/remove the class as needed. Hope this helps you out a bit!
Marked as helpful
@trunglam7
Posted
@elaineleung Thank you for this feedback! I completely missed that.