-
-
Notifications
You must be signed in to change notification settings - Fork 1k
PrimeTime Watchface #2307
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
PrimeTime Watchface #2307
Conversation
Build size and comparison to main:
|
Looks great! I particularly like the current song on the watch face - whether this makes it in or not, any chance we can add the current song to the Digital watch face as well? I would totally use that feature. |
Alright, I've gotten all the checks to pass now. So sorry for having so many commits now! |
You can always run |
- Added new watchface implementation ("Prime") - Formatting and patch cleanup - Fix short ref not set in GitHub builds - Init notification buffer to prevent uninitialized memory reads - Fix nRF SDK download links in Docker and build scripts - Refactor watchfaces to use xTaskGetTickCount() instead of lv_tick_get()
You are my hero!! 😄 |
What do you guys think of mixing Prime and PineTime into PrimeTime? 😆 |
Looks super cool! I think it's different enough to the Digital face that it should be its own one. Could you move the font to the external flash? That should make the watchface much lighter size wise |
- Added new watchface implementation ("Prime") - Formatting and patch cleanup - Fix short ref not set in GitHub builds - Init notification buffer to prevent uninitialized memory reads - Fix nRF SDK download links in Docker and build scripts - Refactor watchfaces to use xTaskGetTickCount() instead of lv_tick_get()
I have no idea how I've done this, but I've accidentally added A LOT of unnecessary files. |
Looks like your tree isn't doing so good 😅 Also when you rebase, it's worth thinking about what you're rebasing onto. I'd usually rebase onto the |
I think I've accidentally done a |
OK, so the first thing to do is make a backup of the branch so you can go back in case something goes wrong after. Here you've got a few options. Start an interactive rebase onto main. You could drop (i.e just remove the line from the list of commits to apply) 6608cf9 and then remake the change again after. Or, you could choose to edit 6608cf9 in the rebase queue. Once the rebase has stopped at the commit, undo the commit ( When you do an (interactive) rebase, each commit gets replayed onto the base you choose. E.g. if you do |
While this sounds very smart and probably would have been the safer way to do it by far, I've somehow managed to just revert the commit and remove the rest by hand. And miraculously that worked and didn't absolutely break everything. I apologize for the very unprofessional work in this PR... 🥲 |
No worries, whatever works 😄 Learning rebasing might be worth it though. Having a nice commit history makes review faster but it's not a must, most of the time I look at the total diff of the PR anyway. Changes look solid and resource usage looks better now :) I'd like to review this soon but will have to see time wise |
Yeah, I'll definitely learn rebasing the next time around! |
Removes unnecessary line from old merge
Hey all!
This PR introduces a new watch face I recently made. The design is based on the existing Digital watch face. Whether this should replace the current Digital face or live alongside it, I don’t have a strong preference either way.
This watch face uses a new font I added called Karnivore. It’s a free TTF font, so it should be fine licensing-wise—but feel free to flag anything I might’ve missed.
One neat addition is that it shows the current song playing directly on the face, below the line showing the day of the week and the date. Though it has to be said that its color might be a little bit dark on the black background, but I thought it looked best like that regardless.
I also styled the heart rate and step count icons/values to match the colors used in the InfiniLink app because I thought they looked nice(r) :)
The day of the week is shown in sentence case (Fri), instead of all caps like on the Digital face. The date is in DD.MM.YYYY format. I personally prefer using periods and leading zeroes for single-digit months.
The battery percentage is shown at the top. Right now it’s hard-coded, but this can of course be updated if PR #1959 is merged.
I think that covers everything! Hope you like it 🙂