Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

More Tooltips #124

Closed
vinnymac opened this issue Aug 24, 2016 · 1 comment
Closed

More Tooltips #124

vinnymac opened this issue Aug 24, 2016 · 1 comment

Comments

@vinnymac
Copy link
Owner

vinnymac commented Aug 24, 2016

Some ideas I have seen around additional information on the table around evolves, cp, and IVs.

https://github.com/wwayne/react-tooltip
https://react-bootstrap.github.io/components.html#tooltips
http://styleguide.cfapps.io/react_base_tooltips.html

I think we should add additional tooltips for all of this info. They appear to work well, and the IVs look kind of ugly right now with the ADS nearby. Thoughts?

@vinnymac vinnymac added this to the v1.4 milestone Aug 24, 2016
@vinnymac vinnymac changed the title Tooltips More Tooltips Aug 24, 2016
@iPaulis
Copy link

iPaulis commented Aug 24, 2016

Agreed. Tooltips are a nice way to show info that otherwise could be visually overwhelming. ADS could go on tooltips when we hover IV%, although as it is right now is also fine for me. In CP the tooltip could show Max CP for that Pokemon. The additional info for candy/evolves would be great in tooltips. Personally, only with a "+12" in the candy column tooltip or a "+1" in the evolve column tooltip would sufice to understand that you need 12 more candy to evolve all the pidgies you have or you have enough candy to evolve 1 more weedle. Anyway, if pokemon/candy exactly match, there shouldnt be any tooltip.

vinnymac added a commit that referenced this issue Aug 25, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants