Merge pull request #16 from elchinatalpf/refactor/app

updated old cities and local storage issue

Author: elchinatalpf

.vscode/settings.json

@@ -0,0 +1,3 @@
+{
+    "liveServer.settings.port": 5501
+}

index.html

@@ -9,7 +9,6 @@
     <link href="https://cdn.jsdelivr.net/npm/[email protected]/dist/css/bootstrap.min.css" rel="stylesheet"
     integrity="sha384-9ndCyUaIbzAi2FUVXJi0CjmCapSmO7SnpJef0486qhLnuZ2cdeRhO02iuK6FUUVM" crossorigin="anonymous">
     <link rel="stylesheet" href="./style.css" />
-
     <script src="https://cdn.jsdelivr.net/npm/[email protected]/dist/js/bootstrap.bundle.min.js"
     integrity="sha384-geWF76RCwLtnZ8qwWowPQNguL3RmwHVBC9FhGdlKrxdiJJigb/j/68SIy3Te4Bkz" crossorigin="anonymous"></script>
     <script src="https://code.jquery.com/jquery-3.7.0.min.js"

script.js

@@ -5,14 +5,19 @@
   const renderCityHistory = () => {
     $("#listofCities").empty();
     cityHistory.forEach((city) => {
-      $("listofCities").prepend(`<tr><td>${city}</td></tr>`);
+      const cityElement = $("<tr>").html(`<td class="city-name">${city}</td>`);
+      cityElement.on("click", ".city-name", function() {
+        const clickedCity = $(this).text();
+        fetchWeatherData(clickedCity);
+      });
+      $("#listofCities").prepend(cityElement);
     });
   };
 
   const addcityHistory = (city) => {
     if(!cityHistory.includes(city)) {
       cityHistory.push(city);
-      localStorage.setItem("cityHistory".JSON.stringify(cityHistory));
+      localStorage.setItem("cityHistory", JSON.stringify(cityHistory));
       renderCityHistory();
     }
   };