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

Wind speed animation #958

Open
papjul opened this issue Apr 17, 2024 Discussed in #313 · 3 comments
Open

Wind speed animation #958

papjul opened this issue Apr 17, 2024 Discussed in #313 · 3 comments
Assignees
Labels
Idea Ideas can become new feature or improvement Priority: medium Issues that are specific to some features/users

Comments

@papjul
Copy link
Collaborator

papjul commented Apr 17, 2024

Discussed in #313

Originally posted by tjsuominen August 6, 2023
I am using Today Weather in parallel with Breezy Weather (Love your app, keep up the great work!).

One thing that I would love to get "adopted" somehow from TW is the animation effect of wind speed (both for the summary and as separate card)

Additional note: TW does not have a seasonal visualization in the wind speed animation.

Why? Because for most people showing the single "Wind x.x" speed does not tell much, but when it gets animated in a neat way, it tells a lot more.

I'm not in favor of "copying" features, but I hope that could be implemented in some neat way.

Any additional thoughts?

@papjul papjul added Idea Ideas can become new feature or improvement Open to contributions This issue is open to the community for implementation Priority: medium Issues that are specific to some features/users labels Apr 17, 2024
@papjul
Copy link
Collaborator Author

papjul commented Apr 17, 2024

Open to contributions, but I'm not sure where we should put this animation, if you have any idea? A dedicated card would be too much IMO

@min7-i
Copy link
Contributor

min7-i commented Apr 17, 2024

For inspiration, the Yr app has a wind speed animation integrated into their wind icon (see top right in the recording below).

yr.wind.mp4

Instead of creating a "large" animation like Today Weather does, we could adapt Yr's approach for our wind icon in the header. Although I'm not sure if I like the idea of having a single animated icon while the rest of the header icons is static.

@papjul
Copy link
Collaborator Author

papjul commented Apr 17, 2024

I had something like that in mind, but I wasn't sure how it would be designed. Nice to see this example!

And it also fixes #738, but our icon is a bit small in comparison.

We could also add it to the hourly details dialog, and daily details activity.

I'm not sure if I like the idea of having a single animated icon while the rest of the header icons is static

It could be animated only when you tap on it, like we do when you tap on the weather icon on hourly forecast.

I will take a look.

@papjul papjul removed the Open to contributions This issue is open to the community for implementation label Apr 17, 2024
@papjul papjul self-assigned this Apr 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Idea Ideas can become new feature or improvement Priority: medium Issues that are specific to some features/users
Projects
None yet
Development

No branches or pull requests

2 participants