@visualdenniss
Posted
Congrats on completing the challenge successfully! Everything looks great and it is responsive as well. It is also great that u used json data to render their heights dynamically.
I noticed that issue with hover as well. Imho it is an overkill to use JS logic to display them on hover as they could be done with css only and using states for that overcomplicates that, not only the code but also introduces bugs as you mentioned.
If you want to see an alternative way, you can check out my solution here, where i tried to use JS for hover states at all, i only calculate the highest value to make them look blue (this is just optional tho) : Live: https://frontendmentor-showcase.netlify.app/components/expense-chart Source Code: https://github.com/visualdenniss/frontendmentorProjects/tree/main/src/components/junior/ExpChart
Hope this feedback was helpful!