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

bmi update #15

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@dleung25
Copy link

commented Dec 11, 2018

I felt like the application was a little bare-bones; having seen some BMI web calculators, I thought it might be nice to have a section where users can see the ranges for themselves, as well as a link to a BMI table:
bminew

@characterdog

This comment has been minimized.

Copy link
Owner

commented Dec 11, 2018

Thanks for your contribution. Just ignore Stickler, it isn't configured correctly.
Actually I want to get rid of weblinks due to privacy concerns. IMO the best solution would be a colored line (like this https://www.prokerala.com/health/health-calculators/bmi.php).

@dleung25

This comment has been minimized.

Copy link
Author

commented Dec 11, 2018

Thanks for your contribution. Just ignore Stickler, it isn't configured correctly.
Actually I want to get rid of weblinks due to privacy concerns. IMO the best solution would be a colored line (like this https://www.prokerala.com/health/health-calculators/bmi.php).

That looks good - maybe something like this?
bmi chart

Also: would applying something like that to the calculator be something to look at (i.e move an arrow to point at where the person is on the scale, after the calculation has been made

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.