background for analog watch face in lvgl#1824
background for analog watch face in lvgl#1824FintasticMan merged 3 commits intoInfiniTimeOrg:mainfrom
Conversation
This commit removes the background image for the WatchFaceAnalog and replaces it with lvgl widgets. It aims to keep the original look.
|
Build size and comparison to main:
|
FintasticMan
left a comment
There was a problem hiding this comment.
Looks good! You should just be able to remove the bg_clock.c file from CMakeLists.txt, then delete the file itself.
FintasticMan
left a comment
There was a problem hiding this comment.
I quite like this with the current colours. It might be nice to keep the 'PineTime' text, but I'm not sure we have the right font to make it look good. I would like some other people's opinions on how it looks before I approve this.
|
I tried it with 'PineTime' first and can confirm that it does not look good with the current font 😄 |
|
It would be interesting what the ram usage at runtime difference between this new and the old one with the background image is |
|
Which number do you need? Memory heap |
|
It would be better to use a label to display the current free memory on the watch face itself, because the face is unloaded when you go to the about page to look at the memory. (assuming you don't have a debugger connected) |
|
Makes sense. How can I get the current free memory? |
|
|
Personally, I would like Fredoka 600 from Google Fonts. |
|
Tbh, I like the watch face more without some brand label. Isn't it also possible (or at least a goal) to run InfiniTime on non-PineTime watches? |
|
You're right, it is possible to run InfiniTime on other watches, currently only variations of the Colmi P8. |
|
Great Job ! |
|
Here are the measurements: |
JF002
left a comment
There was a problem hiding this comment.
This new implementation of the watch face looks good! I think the small visual differences are very much compensated by the amount of freed flash memory!
* change background image to widgets This commit removes the background image for the WatchFaceAnalog and replaces it with lvgl widgets. It aims to keep the original look. * remove comments and background image --------- Co-authored-by: minacode <minamoto9@web.de>

Alternative to #1819.
This commit removes the background image for the WatchFaceAnalog and replaces it with lvgl widgets. It aims to keep the original look.
The refresh is almost instant now.
I am not sure at how much places the background image has to be removed. If someone knows more about that, please have a look at it.
Colors are up to debate 😁