From 856db4599abe299d53a390d87f19382b6e86ceba Mon Sep 17 00:00:00 2001
From: not manx <boku@plum.moe>
Date: Fri, 31 Jul 2020 04:07:31 +0100
Subject: [PATCH] Add comments and cleanup

---
 .gitignore          |  2 +-
 src/App.svelte      |  3 ---
 src/index.svelte    | 14 +++++++++-----
 src/lightbox.svelte |  6 +++---
 4 files changed, 13 insertions(+), 12 deletions(-)

diff --git a/.gitignore b/.gitignore
index 4064c60..36170a7 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,2 +1,2 @@
 node_modules
-public/out
+public
diff --git a/src/App.svelte b/src/App.svelte
index 9bd7b94..208593f 100644
--- a/src/App.svelte
+++ b/src/App.svelte
@@ -1,9 +1,6 @@
 <script>
 	import Router from 'svelte-spa-router'
 	import Index from './index.svelte';
-	const themes = [
-		'default'
-	]
 
 	const routes = {
 		'/': Index
diff --git a/src/index.svelte b/src/index.svelte
index 4daccd1..3d42f5e 100644
--- a/src/index.svelte
+++ b/src/index.svelte
@@ -1,6 +1,7 @@
 <script>
 	import { onMount } from 'svelte';
 	import Lightbox from './lightbox.svelte';
+
 	let file_list;
 	let open;
 	let thumb_list = [];
@@ -16,24 +17,27 @@
 			.then(resp => resp.text())
 			.then(data => data.split('\n'));
 
-		// Loop backwards efficiently
+		// Loop backwards efficiently.
 		for (let i = file_list.length - 2; i >= 0; i--) {
 			const file = file_list[i];
+
+			// Thumbs are only ever .jpg
 			thumb_list.push(file.substr(0, file.lastIndexOf('.')) + '.jpg');
 		}
 	});
 	
 	function open_lightbox(i) {
-		open = null;
+		// If `open' is already `i' updates aren't sent to lightbox.
+		open = null; 
 		open = i;	
 	}
 </script>
 
 {#if file_list}
 	<section>
-	{#each thumb_list as thumb, i}
-		<img on:click={() => open_lightbox(i)} src="{url.root}{url.thumb}{thumb}" />
-	{/each}
+		{#each thumb_list as thumb, i}
+			<img on:click={() => open_lightbox(i)} src="{url.root}{url.thumb}{thumb}" />
+		{/each}
 	</section>
 {/if}
 <Lightbox open={open} url={url} images={file_list} />
diff --git a/src/lightbox.svelte b/src/lightbox.svelte
index b4d5aa7..74be78c 100644
--- a/src/lightbox.svelte
+++ b/src/lightbox.svelte
@@ -10,7 +10,8 @@
 
 	$: image = open !== null && open >= 0 ? set_images()  : '';
 	$: is_open(open);
-	
+
+	// This needs renamed
 	function is_open(i) {
 		if (should_open && i !== null &&  i >= 0) {
 			class_list += ' open'
@@ -85,7 +86,7 @@
     width: 100vw;
     height: 100vh;
     opacity: 1;
-    transition: opacity 1s;
+    transition: opacity 1s; /* This is being applied when we add `slide_classes' */
 		display: flex;
     justify-content: center;
     align-items: center;
@@ -96,7 +97,6 @@
     background-color: rgba(0,0,0,0.7);
     z-index: 0;
   }
-
 </style>
 
 <svelte:window on:keydown={handle_keys} />