msupply-foundation / msupply-cold-chain

Android application for viewing and monitoring temperatures of fridges
GNU General Public License v3.0
3 stars 3 forks source link

Update ui #283

Closed mark-prins closed 9 months ago

mark-prins commented 9 months ago

Fixes #282

Have moved the sensor name to the left, which I think is more logical in a LTR reading anyway. It does divorce the name from the temperature status on the right, but I can cope with that.

The main sacrifice is a little chart height. Is that acceptable?

portrait_2

landscape_2

I've also made the decimal value smaller, because the temperature, when not in breach, was overflowing to two lines. been like that for a little while.. after the previous chart changes.

adamdewey commented 9 months ago

Looks fab @mark-prins - don't think there's a problem with losing vertical space as don't think there will be too many places using 6 sensors on one tablet.

But any chance we could vertically center align the big numbers with the charts?

Was thinking like this:

image image

But no worries if it's too tricky!

mark-prins commented 9 months ago

thanks - I've centered, and aligned, and increased the font size of the breach to be the same as the standard size. That required keeping the sensor name on the left, so I've done that.

For reference, here's how it looks:

l_hot l_low p_hot p_low no-data no-battery