Skip to content
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

Daisy Clock Display 12hrs, 5% ring Increments and Settings to Change the Ring Behavior #3759

Draft
wants to merge 25 commits into
base: master
Choose a base branch
from

Conversation

voloved
Copy link
Contributor

@voloved voloved commented Feb 23, 2025

This commit modifies the Daisy watch behavior by:

  • Making it so the ring goes up in increments of 5%, rather than 10%
  • Allow changing the behavior of the ring; before this commit, it'd only display steps taken throughout the day. The new options are:
    • Show minutes, like the minute hand of an analog watch
    • Show seconds (the watch will update every 5 seconds)
    • Show battery percentage.
  • Added toggle to show time in 12 hr mode.

The ring options and the 12hr mode are both options in the Settings.

The only new concerns from this change are:

  • The Seconds option will drain the watch faster, as it'll update the face every 5 seconds, rather than every minutes
  • The Battery option may cause the Ring to pop due to lack of deadband, As in, if the battery jumps between 50% and 51% continuously, the ring will jump between the two each minute. This is unlikely to be a concern, but I wanted to point it out.

I tested all of the new settings on a Bangle2, and they all work well. The Seconds option confirmed that all of the 5% ring steps looked good as well.

@bobrippling
Copy link
Collaborator

Thanks, LGTM! @hughbarney are you happy for this to be merged?

@voloved
Copy link
Contributor Author

voloved commented Feb 25, 2025

Actually, hang on, I want to make the Sunrise and Sunset time remaining an option on the wheel and make those times show as 12 hours as well.

@bobrippling
Copy link
Collaborator

I'll pop this into draft - if you put it back to ready when you're happy I can look at merging :)

@bobrippling bobrippling marked this pull request as draft February 25, 2025 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants