@elaineleung
Posted
Hi James, great job in working out this React solution! I just had a quick glance at your App.js, and it looks fine to me. In terms of usability, I think it's pretty good too; the only thing I would change is to keep the state of the custom input because I notice that it gets changed to a new value based on which tip button is pressed. Needing to change that value before inputting a new value is a bit unnecessary and can be a slight annoyance.
That's all the comments I have, good work here, and hope to see more! 🙂
Marked as helpful
@Auddity
Posted
@elaineleung Thanks for your reply.
I had the same annoyance with the custom button, however I sort of wrote it off. That's a lesson right there. To not write off something minor like that in terms of user experience.
Cheers.
