Nearly Old School watch face displays wrong hour two hours a day

I noticed a bug last night when trying out the Garmin watch faces. During the midnight hour it displays zero for the hour. I'd guess that this problem will appear during the noon hour as well. It is likely that the code to convert from the 24-hour time to 12-hour time is written as var hour = info.hour % 12, but it should be var hour = 1 + (info.hour + 11) % 12.

Travis

travis.vitek over 10 years ago

Obviously, this may be an issue with the other watch faces. I didn't try them.

travis.vitek over 10 years ago

Yes, it affects the other watch faces as well (at least Timeascii). Additionally, the Timeascii watch face is nearly unreadable on a 920xt because of the text color selection. Using one dark color and one light color might help. I suppose this may be a non-issue if users are allowed to configure the watch face colors via the Garmin Connect App, but the default colors should give a good first impression.

AlphaMonkeyC over 10 years ago

Feel free to leave reviews and ratings (only available if you download an app) on the Garmin app store about the Garmin apps. It's there for this purpose.

-Alpha Monkey

jetsers over 10 years ago

I noticed a bug last night when trying out the Garmin watch faces. During the midnight hour it displays zero for the hour. I'd guess that this problem will appear during the noon hour as well. It is likely that the code to convert from the 24-hour time to 12-hour time is written as var hour = info.hour % 12, but it should be var hour = 1 + (info.hour + 11) % 12.

Travis

Thanks for the info, I haven't had a chance to see that live yet, and just fixed my personal one. Great catch.

travis.vitek over 10 years ago

Feel free to leave reviews and ratings.... It's there for this purpose.

I had originally wanted to post the comments there, but I didn't want to ding the developer publicly (most 920XT users are more likely to see a review there than they are to see a post here...). I'll post a review though.