Nice, if you wanna amp it up, then adding a transition to the color changes is a really nice effect, and relatively easy, effect i have seen for these kind of toggles.
Another way of doing this would be toggling a class with element.classList.
It is a bit more dynamic, and leaves less styling floating in the JS.
Unless you are into CSS-in-JS but that is a bit more advanced compared to this. :)
We're a place where coders share, stay up-to-date and grow their careers.
We strive for transparency and don't collect excess data.