alex-willi / Weather-App

0 stars 1 forks source link

Grow Issue #21

Open alex-willi opened 1 year ago

alex-willi commented 1 year ago

I wish we had been able to implement geocoding into our weather app. The entire app feels arbitrary with out being able to change the location for the forecast. There is no code for geocoding.

The today forecast should include hourly forecast. Instead we included a current weather icon which I think is too small, this is due to css problems. We also put in wind speed to try and fill content for today's forecast. Code for Today:

<div className="window">
  <div className="today">
    <h1 className="todayLow"></h1>
    <h1>Date Today: {data.daily.time[0].slice(5)}</h1>
    <h1> Low: {data.daily.temperature_2m_min[0]}F </h1>
    <h1 className="todayHigh">
      {" "}
      High: {data.daily.temperature_2m_max[0]}F
    </h1>
    <h1 className="todayRain">
      {" "}
      Inches of rain: {data.daily.precipitation_sum[0]}"{" "}
    </h1>
    <h1>Wind Speed MPH: {data.daily.windspeed_10m_max[0]}</h1>
  </div>
</div>

); }

In summery the app is incomplete in my eyes because of no geocoding and I need to get better at css because in situations like this, data falls through the cracks because of style. (weather icon)