I really liked this watch, so I forked it on Github.
Changes:
(1) I have a preference for "Weekday MM-DD" for date so I changed that.
(2) Rebased the full moon computation on the first full moon of 2013, in the Los Angeles (UTC-8, PST) timezone. I documented it in my code.
(3) There was some spacing and frame size issues that obscured the "A" or "P" when in 12-hour mode which I tidied up.
(4) handle_deinit() - Lots of watchfaces don't clean up their resources (bitmaps and custom fonts) when handle_deinit is called. I had to fix my other faces too. This bug causes the font corruption when watch faces are switched quickly.
Thanks for your fixes and suggestions! I was already working on the resources cleaning up, but your approach is better than mine by far!
I had to cherry pick your changes, as I already did some layer stuff improvements. Also, I didn't pick the date format, as is something personal. I was thinking about use a script for compile different versions depending on the date format, but I don't think is worthy right now, is much faster just recompile the watchface. Hopefully, the Pebble team will implement a date format configuration or something like that.
I hope the team also implements some kind of geolocation function to be able to do clever things like sunrise, sunset, etc (I already have icons for that in my font, just in case)
Regarding AM/PM, sorry about that, I should have tested that better, but I always use the 24h format... Anyway, I moved the text from the right edge to below the hours, as it looks great and we have much more free room there, let me know what you think about it.
Thanks for cherry picking, I'm flattered. I was working on cleaning and updating code and will have a pull-able result shortly. It will pull all of the changes out with #define so that they can be turned off. I'm supporting DD-MM as well as MM-DD.
I have rebased my code against yours and am tidying and #define-ing. I'll issue a pull request when it's all done. Then you can grab the changes without cherry picking and compile any combination of options for month, weekday, date format...
Updated using Ricardo's new am/pm layout but with options to show weekday, month, and day. All updates are definable and the defaults in the Github repo are set to produce Ricardo's original.
Comments