InfiniTimeOrg
PTS: add seconds display
After receiving a PineTime for Christmas, I started trying to code a bit for it. The first thing I noticed was that the very nice PineTimeStyle watch face does not show the seconds, so I changed that. The design is adapted from the original Pebble TimeStyle, where the seconds are also displayed in the bottom right corner. I therefore had to move the step counter up and to avoid coming to near to the calendar, I also changed the step counter to be a half circle instead of a full circle.
I am using the very same changes (but applied to master) since more than a week now on my PineTime and it works perfectly fine, but since I only have a sealed version, I didn't dare to flash the develop branch with the changes. However, I now also simulated it using the awesome work of @NeroBurner in https://github.com/NeroBurner/InfiniTime/tree/lv_simulation_lvgl7 on the current develop branch and everything seems to work here as well.
This is how it looks like:
https://user-images.githubusercontent.com/18088991/148376673-56090395-fa88-4022-b68f-e6dc4a500719.mp4
Also mentioning @kieranc since he is the original author of the PineTimeStyle watchface. Would be really nice to have this included in the official InfiniTime builds!
Nice work, it looks good and the half circle gauge is pretty nifty.... However, this needs to be a configurable option, I don't think it should be the default for everyone. I'm working on getting some weather data into the watchface currently but at some point I will need to add additional settings for which items are displayed in the sidebar, a seconds display can certainly be one of them!
@kieranc Excited to hear about weather info being integrated into the watch face! In case you would like some help on that or on the settings screen, I'd be happy to provide it!
I quite like the look of this and it also makes the gauge easier to understand. Can you fix the conflicts, so we can consider merging this @ialokim?
I struggled a bit with the docker build environment: building the docker file locally resulted in InfiniTime build failures until I used the latest version of the Dockerfile from develop. Afterwards, I was not able to build InfiniTime on current develop, as it fails with several fatal error: date/date.h: No such file or directory errors.
I did however apply the same changes to the last released version 1.10.0 where I was able to build InfiniTime. That version is running smoothly on my PineTime now.
missing date/date.h could be because of the introduction of the date library as a git submodule and may be fixed by running git submodule update --init --recursive
https://github.com/InfiniTimeOrg/InfiniTime/pull/1183
Yes, that was it, obviously! Now I can build this branch as well locally. Thanks @NeroBurner!
I'm still not keen on this being implemented as a default, I personally don't want it, I have no problem with it being implemented with a selector to choose between this and the current default (which I'm working on) but as a replacement for the current gauge, I'm firmly opposed.
This is probably a matter of taste and an artistic choice, but, as @kieranc is the original author of this watchface and continues to maintain it, I would not like to go against his opinion.
@kieranc Keep us up to date when you have something to show us regarding the selector :)
I've got this far, adding an options settings button in addition to the color settings, which so far allows toggling between full step gauge, half gauge and seconds or numerical step count. Todo, implement an additional persistent setting to save the selected option and pretty it up a bit. I'll keep going when I can!