@Nam-Hai
Posted
The accordion menu works fine, so great work for your first encounter with JavaScript.
However, in most cases, accordions menu are closed by default on websites. Futhermore, you could have given the "buttons" the attribute cursor: pointer;
so the user knows it can interact with them.
Also, when every accordions are opened, some go off the card which makes the design feel cheap. You could have either :
- make the card gain height to contain everything when too many menus are opened.
- authorizes only open one menu at a time, and than opening one close the one that was already opened. (I hope i'm clear xD)
Marked as helpful
@Art-wdt
Posted
@BlueTompon Hey!
Thanks for your helpful advices.
I will work on this in the future. I completely forgot about the fact that the accordion is closed by default