@elaineleung
Posted
Hi Paulina, well done on this challenge and writing out all that JS!
This is more of a UX comment: At first I thought the amount tags weren't working, but then I looked at your code and realized you wrote them as click events. I think that unless there had been some explicit instruction (such as, "click to see amount"), I really would not have known the bar is meant to be clicked on for the amount to show up (plus I also worked on this challenge and know the brief). Also, since there is already some change seen when a user interacts with the bar, as in, hovering over the bar changes its color, there might not be a thought that there's more interaction. In terms of UI, I think you can try adding a bit more padding around the amount tags.
Once again, great work and looking forward to more 🙂
@testerium
Posted
@elaineleung Hi Elaine! Thank you for your comments ! :) You're absolutely right that from the user's perspective it's not clear that he should click on the bar to see the amount. Padding can indeed be enlarged. I will try to improve it. Thanks!