InfiniTimeOrg / InfiniTime

Firmware for Pinetime smartwatch written in C++ and based on FreeRTOS
GNU General Public License v3.0
2.7k stars 923 forks source link

New dynamic watch face: Horizon #1396

Open Cesura opened 1 year ago

Cesura commented 1 year ago

Preview

watch

New resources

(Public domain/GPL, https://www.dafont.com/pinecone.font)

Description

Hi! First of all, thank you for this project; you're doing some great work here.

I've created a somewhat minimalistic watch face which changes color throughout the day based on the current hour, using warm colors in the daytime and cooler colors in the evening.

The watch face itself has the following components:

The font rendering at 1 bpp made for some unclean edges with certain sizes, so I had to play around to get ones that were more or less consistent. Also, I'm not particularly attached to the name, so it can be changed if a more suitable one comes to mind.

I double checked everything with clang-{tidy,format}, but please let me know if I've still managed to violate some parts of the style guide. :smile:

minacode commented 1 year ago

It is beautiful 😊 But (sorry..) wouldn't it be better to switch the warm and cold colors? I am thinking of all the redshift implementations out there. Blue light makes you awake.

Cesura commented 1 year ago

@minacode It's a reasonable point, yeah. Although I don't know if people check their watches often enough for the colors to have any meaningful impact on their sleep? Plus the evening colors are already quite purple in nature. :smile:

But if someone would like to play around with the color scheme, I'm also not opposed.

LinuxinaBit commented 1 year ago

The sunset 🌅 vs midday 🏞 light does turn redder at night for various atmosphere reasons, so it could actually make more sense if midday was bright blue and midnight was reddish?

Just maybe a lighter shade of blue would be more appropriate for that color scheme.

It still looks really good as is, and I kinda like it better this way, just the blue light thing could be worth it?

Cesura commented 1 year ago

If there are suggestions for a change to the color scheme, please post an array of new hex colors similar to the one defined in src/displayapp/screens/WatchFaceHorizon.h, and perhaps also screenshots for us to make a simple visual comparison.

Otherwise, this is the build I am submitting to be merged, and I personally don't see any reason for it to be blocked unless alternatives are provided.