Hana• 870
@Hanka8
Posted
Hi Jimmy, nice job!
My suggestions:
• You can add multiple classes in "element.classList.add()" or "element.classList.remove()", so your JS code will be a little bit shorter;
• when there is no text content in the <button> tag, you should add the "aria-label" to label the element or "aria-labelledby" referencing to some other element with accesible name;
• there are no margins around the content in the design;
Other than that your solution is great, keep up the good work!
0
