nightscout / AndroidAPS

Opensource automated insulin delivery system (closed loop)
https://wiki.aaps.app
GNU Affero General Public License v3.0
696 stars 1.7k forks source link

[DEV] Status Line is not showing all information #2448

Closed triplem closed 1 year ago

triplem commented 1 year ago

The Statusline shows the Date of the last reservoir change as well the the age of the Battery, but not it's values. On the Action Screen only parts of the value are shown. In Landscape Mode this seems to be working fine.

I would prefer of the value is shown before the age, ao to be able to show the value instead of the age of both.

triplem commented 1 year ago

The Phone is a OnePlus 8T.

MilosKozak commented 1 year ago

screenshot?

triplem commented 1 year ago

Screenshot_2023-03-04-10-15-28-23_3eaa31bb2c2a38a1c605d9dc918cf694.jpg

Screenshot_2023-03-04-10-15-16-06_3eaa31bb2c2a38a1c605d9dc918cf694.jpg

Screenshot_2023-03-04-10-15-11-23_3eaa31bb2c2a38a1c605d9dc918cf694.jpg

Screenshot_2023-03-04-10-17-30-80_3eaa31bb2c2a38a1c605d9dc918cf694.jpg

MilosKozak commented 1 year ago

this doesn't have good solution. formating is done by android

vanelsberg commented 1 year ago

Screen look good for me. Think is is phone or phone settings specific, maybe check phones display setting like resolution en/or font size?

swissalpine commented 1 year ago

If you select "Skin: Low resolution skin" in the settings, then the superfluous, space-eating labels like "Alter:" and "Abgegeben:" are hidden. In the past (some years ago), these labels did not exist either. And since the information is actually self-explanatory, we could basically say goodbye to it again. Or @MilosKozak?

vanelsberg commented 1 year ago

If you select "Skin: Low resolution skin" in the settings, then the superfluous, space-eating labels like "Alter:" and "Abgegeben:" are hidden. In the past (some years ago), these labels did not exist either. And since the information is actually self-explanatory....

Yes, translations may give problems, like in this case. I think for most this is not a problem. How about just showing icons, like on main overview, pump/reservoir/sensor?

triplem commented 1 year ago

Well, setting "Low Resolution" does work. But eg "Abgegeben" is then still shown.

# Screenshot_2023-03-09-18-33-12-85_3eaa31bb2c2a38a1c605d9dc918cf694

Also this does not really has an effect on the overview

Screenshot_2023-03-09-18-35-21-22_3eaa31bb2c2a38a1c605d9dc918cf694

Still I think that the level in the reservoir is more important then the age. Could this be swapped? Same is probably true for the battery level (well, at least partly).

Philoul commented 1 year ago

Still I think that the level in the reservoir is more important then the age. Could this be swapped? Same is probably true for the battery level (well, at least partly).

On low res screen, there are less information than on higher res:

With my phone for ex, I have age and level for reservoir, age and level for battery

Screenshot_20230326-172552_AAPS.jpg

But it's a good idea to keep levels and hide ages on low res...

aliswalker commented 1 year ago

@Philoul without going OT as this does affect status lights on low res – Do you know why your COB doesn't display a timestamp like mine/OP? It's on a new line which for low res users means scrolling with status lights on.

LowRes

Yours both also show TBR % vs U/h which was proposed as a setting under #2200

Philoul commented 1 year ago

1) The time below COB value is only visible in dev branchs, not in master (I just hide it on my own version).

2) The format of TBR is directly linked to the pump driver and how the pump manage this information:

So if your pump driver send to your pump an absolute rate when a tbr is set, then you will see the absolute rate in overview... (I think it's to be able to check directly with your pump UI if both informations are consistant)

MilosKozak commented 1 year ago

https://github.com/nightscout/AndroidAPS/commit/6f22c5122ffd50b9c3ff19d91b60ec67d59b10a9