Infineat watch face is missing heart rate monitor icon/reading.

InfiniTimeOrg / InfiniTime

Firmware for Pinetime smartwatch written in C++ and based on FreeRTOS

GNU General Public License v3.0

2.75k stars 941 forks source link

Infineat watch face is missing heart rate monitor icon/reading. #1457

Closed apoorv569 closed 1 year ago

apoorv569 commented 1 year ago

Verification

[X] I searched for similar bug reports and found none was relevant.

What happened?

The heart rate icon/reading is missing.

What should happen instead?

Heart rate icon/reading should appear as in other watch faces.

Reproduction steps

Change the watch face to Infineat.

More details?

No response

Version

v1.11

Companion app

Gadgetbridge

JF002 commented 1 year ago

This could be the result of an artistic choice by the author of the watchface. @dmlls, what do you think?

dmlls commented 1 year ago

@apoorv569 @JF002 Hi, it was indeed a design choice, briefly discussed here: https://github.com/dmlls/InfiniTime/pull/3#issuecomment-1072860833.

In any case, if users think this is a must, I'm open to reconsidering :)

apoorv569 commented 1 year ago

@apoorv569 @JF002 Hi, it was indeed a design choice, briefly discussed here: dmlls#3 (comment).

In any case, if users think this is a must, I'm open to reconsidering :)

I see. Well we can definitely wait for some feedback from other users.

InFerYes commented 1 year ago

I find the HR monitor to be more useful if it can be read from the watch face.

JF002 commented 1 year ago

Thanks for your feedback @dmlls !

@apoorv569 If you don't mind, I'll move this post to a discussion to let everyone interested can give their opinion.