InfiniTime
InfiniTime copied to clipboard
InfiniTimeOrg
Watchface: add weather option to Casio style watchface
Adds the option to turn one of the segments on the Casio watchface into weather.
Build size and comparison to main:
| Section | Size | Difference |
|---|---|---|
| text | 374692B | 1604B |
| data | 948B | 0B |
| bss | 22544B | 8B |
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
Das PR does not apply anymore due to the recent changes in the weather subsystem - would be cool to have this updated against the current state of main.
Updated my watch and it works like a charm as it did before. Thanks.
Would be nice to see this included in 1.16.
Would it be possible to choose which segment is replaced by weather?
I made a hardcoded version of your idea as well a while back, but I replaced WeekNumber instead. If you could implement a way to choose which segment is replaced, we could essentially merge those ideas into one big PR.
Would it be possible to choose which segment is replaced by weather? I made a hardcoded version of your idea as well a while back, but I replaced WeekNumber instead. If you could implement a way to choose which segment is replaced, we could essentially merge those ideas into one big PR.
Yeah I think I can do that, doesn't sound too difficult.
For reference, I put the weather icon to align top left at 10, 22 and the temperature to align top left 45, 22 Feel free to just copy these or take them as a reference point for improving the looks of it.
Oh I might even like that more than my approach! Good job, will definitely be using this, at least once 1.15 rolls out 😁
For the segment that replaces the weeknumber, you should center the temperature and weather icon based on the weekday and offset them vertically (in the negative direction of course, since you're going up on the display).
Also, you merged something about notif vibration strength. I don't think you meant to put this into the PR.
Other than that, I'd love to have this in 1.16.0!
Oops sorry yeah didn't mean to commit the vibration stuff to this branch, I'll fix that when I get the time.
Oops sorry yeah didn't mean to commit the vibration stuff to this branch, I'll fix that when I get the time.
I came across this "accidental addition" and i really like it. Would be cool if you'd provide an additional PR for this.
Sorry it took so long, but I've finally removed the unrelated commits. I'll get to fixing the alignment later!
The merge box thingy is telling me one of the reasons merging is blocked because the branch "must not contain merge commits". Is this a real requirement for getting this merged?
Sorry it took so long, but I've finally removed the unrelated commits. I'll get to fixing the alignment later!
About the alignment, you can check my branch with the extremely long name of "1.15_ANCS_HR_CASIO_DGTL-W" where I think I nailed the alignment perfectly.
Here's a few pics:
I've been having some issues getting the simulator running on my setup (trying to run from a dev container), but I'll hopefully fix that and then this PR this weekend.