wger icon indicating copy to clipboard operation
wger copied to clipboard

Published 20 hours ago • verified publisher icon wger-project

Add correct units for bmi calculator and weight entries

Open jiany18 opened this issue 3 years ago • 1 comments
trafficstars

Proposed Changes

  • When I was using the web app, I was really confused about which units the BMI calculator is using for weights
  • Added units to weight in BMI calculator and weight entry to avoid future confusion

jiany18 avatar May 06 '22 02:05 jiany18

Thanks for the PR! There are actually a couple of places where we do consider whether the user uses metric or imperial units, but yeah, it would be better to explicitly display the units and change them according to the profile settings. The BMI calculator actually is aware of this and will convert the values accordingly in calculate_bmi()

rolandgeider avatar May 06 '22 14:05 rolandgeider