Akylas / oss-weather

An OSS weather app for iOS/Android
MIT License
161 stars 13 forks source link

Main page new chart doesn't show line for wind speed? #175

Closed 0-5-0 closed 1 month ago

0-5-0 commented 1 month ago

Should the new main page chart show a line for wind speed? I can tap the chart on the main page to see what the wind is but I might have expected that wind would show up like it does under Settings -> Chart.

farfromrefug commented 1 month ago

@0-5-0 i did that on purpose. the reasoning is that for now i dont feel like the line for wind is useful. I feel like i need a better way to represent it

0-5-0 commented 1 month ago

Ok, I'm still wondering what this new chart format provides over the normal chart/"old" hourly view. The normal one does have the wind speed under each weather icon, the temperature, if it's just for being able to see further ahead on one screen the original hourly view could be configurable so we can choose between 1 hour steps or 3-4 hour steps for the time/date scale?

Unless you have bigger plans for the new chart, or maybe having it replace the "old" hourly view, maybe it gives you more flexibility?

farfromrefug commented 1 month ago

@0-5-0 it is very different in the sense that you can see more hours at the same time. Which was the requirement. But in the meantime it is a lot more cluttered and i cant easily show as much data as i show in the old view. I personally much prefer the old view but i know some people want the chart. This is why now both exist

0-5-0 commented 1 month ago

It's not possible to have the old chart be configurable where we could select having it display 3 hour intervals instead of 1 hour? I'm thinking to avoid duplication so you're not having to maintain multiple charts.

farfromrefug commented 1 month ago

@0-5-0 could be possible. But i dont mind managing both. It is cool to have people be able to customize the app to their need.