Skip to content
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

Watchface: add weather option to Casio style watchface #2099

Open
wants to merge 30 commits into
base: main
Choose a base branch
from

Conversation

Ghoelian
Copy link

@Ghoelian Ghoelian commented Aug 4, 2024

Adds the option to turn one of the segments on the Casio watchface into weather.
InfiniSim_2024-08-04_230833
InfiniSim_2024-08-04_230835
InfiniSim_2024-08-04_230942

Copy link

github-actions bot commented Aug 4, 2024

Build checks have not completed. Possible reasons for this are:

  1. The checks need to be approved by a maintainer
  2. The branch has conflicts
  3. The firmware build has failed

@rnwgnr
Copy link

rnwgnr commented Nov 21, 2024

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.

@Ghoelian
Copy link
Author

Ah alright, I'll look into getting it updated when I get the time.

@Ghoelian
Copy link
Author

I've updated my branch, seems to be working on my watch.

@rnwgnr
Copy link

rnwgnr commented Nov 22, 2024

Updated my watch and it works like a charm as it did before. Thanks.

Would be nice to see this included in 1.16.

@tituscmd
Copy link
Contributor

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.
image

@Ghoelian
Copy link
Author

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. image

Yeah I think I can do that, doesn't sound too difficult.

@tituscmd
Copy link
Contributor

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.

@Ghoelian
Copy link
Author

InfiniSim_2024-11-25_203125
InfiniSim_2024-11-25_203132

Added these two options.

@tituscmd
Copy link
Contributor

Oh I might even like that more than my approach! Good job, will definitely be using this, at least once 1.15 rolls out 😁

@mark9064 mark9064 added the enhancement Enhancement to an existing app/feature label Nov 28, 2024
@tituscmd
Copy link
Contributor

tituscmd commented Dec 9, 2024

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!

@Ghoelian
Copy link
Author

Ghoelian commented Dec 9, 2024

Oops sorry yeah didn't mean to commit the vibration stuff to this branch, I'll fix that when I get the time.

@rnwgnr
Copy link

rnwgnr commented Dec 23, 2024

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement to an existing app/feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants