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

Calendar date number repeats upon month changing #44

Closed
BrentTechDesigns opened this issue Oct 25, 2020 · 4 comments
Closed

Calendar date number repeats upon month changing #44

BrentTechDesigns opened this issue Oct 25, 2020 · 4 comments

Comments

@BrentTechDesigns
Copy link

I noticed this morning (October 25 2020) that the calendar dates displayed:
18 19 20 21 22 23 24
[25] 26 27 28 29 30 31
1 1 2 3 4 5 6

This appears due to the calendar month rotating from a month ending with 31 days on a Saturday. The big was easily remedied by going to another screen and coming back to the watch face. The calendar properly set to:
18 19 20 21 22 23 24
[25] 26 27 28 29 30 31
1 2 3 4 5 6 7

@arromdee
Copy link

arromdee commented Oct 31, 2020

This issue is not caused by 31 day months. It is caused by daylight savings time.

The program finds the next day by adding and subtracting multiples of 24 hours. If the time is between midnight and 1 AM, and the clocks are being set back, 24 hours after 12:01 AM Sunday is 11:01 PM still on Sunday, Since the day starts on midnight and the calendar refreshes at the start of the day, this means that the calendar goes bad at midnight and stays that way until it passes 1 AM and it's refreshed (by going to another screen and changing to the watch face.)

I have a patch for this as well as for making Sundays, Saturdays, and/or US Federal holidays red based on a setting. I have no idea what to do with it--should I mail it to the maintainer? Attach it here?

@mattrossman
Copy link
Owner

@arromdee good catch! You can submit your patch as a pull request against the develop branch of this repository. I'll review it and merge it in once everything looks good.

@arromdee
Copy link

I submitted a patch which 1) fixes this bug and 2) adds the requested ability to make Sundays red (along with Saturdays and US Federal holidays).

mattrossman added a commit that referenced this issue Nov 2, 2020
@mattrossman
Copy link
Owner

Thanks for the patch, it's included in v1.15.0

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

No branches or pull requests

3 participants