Watchface: Numerals - #2047
Watchface: Numerals#2047liamcharger wants to merge 1 commit into
Conversation
|
Build size and comparison to main:
|
mark9064
left a comment
mark9064
left a comment
There was a problem hiding this comment.
Looks great from the screenshot! I'll give this a test run after any changes from feedback are in
|
I'll test this on my PT shortly and review after |
mark9064
left a comment
mark9064
left a comment
There was a problem hiding this comment.
The font ranges at the moment are very large, and the watchface cannot load (too much memory)
With these changes it works and everything seems as expected 😄
|
Gotcha, I'll make a commit when I have the chance! |
mark9064
left a comment
mark9064
left a comment
There was a problem hiding this comment.
Sorry for the slow review! I'm very busy, and there are many PRs. I think this is almost there!
|
Are there any plans to add this to the main firmware? I would love this as my watchface. |
|
Waiting for @liamcharger to respond on review still. I think it's a good fit for InfiniTime :) |
|
Yep, I’ve just been busy :) I fixed the conflicts and updated the branch with the GitHub UI, I’ll break into the code soon to finish the review :) |
|
When can we expect to see a 1.16.1 build with this watchface? (or maybe 1.17.0, I don't know the plan) |
|
It'll definitely won't be before 1.17. We would only push 1.16.1 update if there was a critical bug. Anyways 1.17 would be a good bet. It just depends on how fast I finish out the review and what the maintainers decide to merge |
8dbd8f9 to
0b9d14a
Compare
|
Whew finally done with the review @mark9064 ! |
4 participants
This PR adds a Numeral watch face, with inspiration from Apple Watch's numeral watch face. An image is attached below.
