Conversation
apradoada
left a comment
apradoada
left a comment
There was a problem hiding this comment.
Even though this isn't exactly the version of the html/javascript combos we ended up doing, I can tell y'all have a really good grasp of both html and javascript!
| <header class="header"> | ||
| Weather Report <span>for the lovely city of | ||
| <span id="headerCityName"> | ||
| Seattle |
There was a problem hiding this comment.
This is totally fine but remember that you could keep this empty and then load the default state when you begin!
| <div class="temperature_content"> | ||
| <div class="change_temperature"> | ||
| <span id="increaseTemp">⬆️</span> | ||
| <span id="temp">70</span> |
There was a problem hiding this comment.
Same thing as above! This could be loaded in from the default state!
|
|
||
|
|
||
| <script src="./node_modules/axios/dist/axios.min.js"></script> | ||
| <script src="/src/index.js"></script> |
There was a problem hiding this comment.
Overall, your html looks really good here! Well done!
| lat: 47.6062, | ||
| lon: -122.3321, | ||
| }; | ||
|
|
There was a problem hiding this comment.
Good job creating a default state here!
| state.temp -= 1; | ||
| changeColorAndGarden(); | ||
| }; | ||
|
|
| const cityInput = document.getElementById('cityNameInput'); | ||
| cityNameInput.value = 'Seattle'; | ||
| modifyCityName(); | ||
| }; |
| .catch((error) => { | ||
| console.log('Error calling OpenWeather'); | ||
| }); | ||
| }; |
There was a problem hiding this comment.
Great job with these axios functions. The move to make them async functions was a really smart choice!
|
|
||
| getWeather(); | ||
| const getTemp = document.getElementById('getTemp'); | ||
| getTemp.addEventListener('click', getWeather); |
3 participants
No description provided.