-
-
Notifications
You must be signed in to change notification settings - Fork 1k
Remove leading zero for HRM value #2309
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
base: main
Are you sure you want to change the base?
Conversation
Alarm: Simplify alarm alerting screen (InfiniTimeOrg#2211)
Build size and comparison to main:
|
Should we change the default |
Or just one? I think, I am fine with any of the variants. |
I think two digits "--" would be the most logical approach here, because that's the amount of digits your heart rate is going to be most of the time. |
Okay, let's try two and get this merged! 🙂 |
Hello everybody!
This is another simple PR of mine.

This PR removes the leading before the BPM value in the Heart Rate app. Previously it would say "083", but now it looks like this:
I think this is cleaner than with the leading zero, but it might be up to preference.
Let me know what you think!