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

Fixed 12-hour clock #1

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Fixed 12-hour clock #1

wants to merge 5 commits into from

Conversation

franc6
Copy link

@franc6 franc6 commented Dec 21, 2017

For a 12-hour clock, the hour should show as a single digit for hours 1 - 9,
not as two digits.

Tom Francis added 5 commits December 21, 2017 08:44
For a 12-hour clock, the hour should show as a single digit for hours 1 - 9,
not as two digits.
Added indicator if quiet time is enabled.
This reverts commit 8fbdf4c.

That commit was made on the wrong branch.
The removal of . was not intentional.  It should remain.  However, escaping .
in a character class is not necessary, so I didn't restore the double \ in
front of it.
If both power saving options are off, the watchface will crash when unloading,
because it tries to unsubscribe from the health service events, but it never
subscribed in the first place!

Also fixed unsubscribing from the tick timer service, since the wrong parameter was passed to unsubscribe when unloading the window, and it wasn't protected by a NULL pointer check (it shouldn't be NULL, but just in case, it's better to check).
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.

1 participant