Merge pull request #14 from davwheat/patch-1

Reverse journey list properly and use proper local time
t5r7 · Jun 5, 2023 · 7c799eb · 7c799eb
2 parents 9c331c3 + d273a37
commit 7c799eb
Show file tree

Hide file tree

Showing 4 changed files with 20 additions and 11 deletions.
diff --git a/common.css b/common.css
@@ -74,6 +74,15 @@ section {
 	padding: 1em;
 }
 
+ol.reversed {
+  display: flex;
+  flex-direction: column-reverse;
+}
+
+ol.reversed li {
+  flex: 0 0 auto;
+}
+
 /* mobile rules */
 @media screen and (max-width: 720px) {
 	header nav {

diff --git a/js/1find-station.js b/js/1find-station.js
@@ -4,7 +4,8 @@ const stationList = document.getElementById("station-list");
 const departureStation = document.getElementById("departure-station");
 const date = document.getElementById("date");
 
-date.value = new Date().toISOString().slice(0, 16);
+// remove last char
+date.value = new Date(Date.now() + (-1 * new Date().getTimezoneOffset() * 60000)).toISOString().slice(0, 16);
 
 departureStation.addEventListener("input", stepOneinputsChange);
 date.addEventListener("input", stepOneinputsChange);

diff --git a/js/localstorage.js b/js/localstorage.js
@@ -10,10 +10,11 @@ if (journeys.length === 0) alert("I couldn't find any journeys in localStorage!
 
 console.log(journeys.length);
 
-let index = 0;
 let outputHTML = "";
 
-for (const j of journeys.reverse()) {
+for (const i in journeys) {
+	const j = journeys[i];
+
 	// thank you, chatgpt:
 	// "edit this piece of javascript so that all of the
 	// "included ${} bits are 'Unset' if the variable doesnt exist"
@@ -22,15 +23,13 @@ for (const j of journeys.reverse()) {
 				<p><span>${j.incomplete ? 'INCOMPLETE' : ''}${j.uploaded ? 'ON RAILMILES' : ''}</span><span class="id">${j.identity ? j.identity : "Identity Missing"}</span> from <b>${j.origin}</b> to <b>${j.destination}</b> (${j.route})</p>
 				<p><span>${j.departureTimeActual}</span> - <span>${j.arrivalTimeActual}</span></p>
 
-				<button onclick="location.href = 'addedit.html?operation=edit&index=${index}'"><span class="emoji-icon">✏️</span> Edit this Journey</button>
-				<button onclick="location.href = 'addedit.html?operation=edit&index=${index}&quickset=dep'"><span class="emoji-icon">⌚</span> Quick Set Departure to Now</button>
-				<button onclick="location.href = 'addedit.html?operation=edit&index=${index}&quickset=arr'"><span class="emoji-icon">⌚</span> Quick Set Arrival to Now</button>
-				${j.uploaded ? "<!--" : ""}<button onclick="location.href = 'addedit.html?operation=upload&index=${index}'"><span class="emoji-icon">☁️</span> Upload to RailMiles</button>${j.uploaded ? "-->" : ""}
-				<button onclick="location.href = 'addedit.html?operation=edit&index=${index}&delete=true'"><span class="emoji-icon">❌</span> Delete this Journey</button>
+				<button onclick="location.href = 'addedit.html?operation=edit&index=${i}'"><span class="emoji-icon">✏️</span> Edit this Journey</button>
+				<button onclick="location.href = 'addedit.html?operation=edit&index=${i}&quickset=dep'"><span class="emoji-icon">⌚</span> Quick Set Departure to Now</button>
+				<button onclick="location.href = 'addedit.html?operation=edit&index=${i}&quickset=arr'"><span class="emoji-icon">⌚</span> Quick Set Arrival to Now</button>
+				${j.uploaded ? "<!--" : ""}<button onclick="location.href = 'addedit.html?operation=upload&index=${i}'"><span class="emoji-icon">☁️</span> Upload to RailMiles</button>${j.uploaded ? "-->" : ""}
+				<button onclick="location.href = 'addedit.html?operation=edit&index=${i}&delete=true'"><span class="emoji-icon">❌</span> Delete this Journey</button>
 			</li>
 			`;
-
-	index++;
 }
 out.innerHTML = outputHTML;
 

diff --git a/localstorage.html b/localstorage.html
@@ -71,7 +71,7 @@ <h2>Manage Journeys</h2>
 	</section>
 
 	<section>
-		<ol id="out">
+		<ol id="out" class="reversed">
 
 		</ol>
 	</section>