From 84ba18e15062a5b9980a8ccfc8216c4bf6d52661 Mon Sep 17 00:00:00 2001
From: Theophyl Hoguet <theophyl.hoguet@etu.u-bordeaux.fr>
Date: Thu, 16 Mar 2023 01:34:03 +0100
Subject: [PATCH] caching + remove "large" div

---
 frontend/src/components/Gallery.vue |  4 +--
 frontend/src/components/Image.vue   | 49 ++++++++++++++++++++++-------
 2 files changed, 39 insertions(+), 14 deletions(-)

diff --git a/frontend/src/components/Gallery.vue b/frontend/src/components/Gallery.vue
index 579e8e9..9871951 100644
--- a/frontend/src/components/Gallery.vue
+++ b/frontend/src/components/Gallery.vue
@@ -19,7 +19,7 @@ export default defineComponent({
 	<div class="flex" v-if="images.keys !== null">
 		<div v-for="[id, image] in images" :key="id" class="image">
 			<label>{{ image.name }}</label>
-			<Image class="gallery" :id="id" />
+			<Image class="gallery" :id="id" :images="images" redirect="home" />
 		</div>
 	</div>
 	<h1 v-else>No image found</h1>
@@ -56,7 +56,7 @@ label {
 }
 </style>
 <style>
-.gallery img {
+.gallery.imageContainer img {
 	margin: 5px;
 	max-width: max(33vw, calc(90vw/v-bind('images.size')));
 	max-height: max(33vh, calc(90vh/v-bind('images.size')));
diff --git a/frontend/src/components/Image.vue b/frontend/src/components/Image.vue
index 414a912..9bf0d7c 100644
--- a/frontend/src/components/Image.vue
+++ b/frontend/src/components/Image.vue
@@ -1,6 +1,7 @@
 <script setup lang="ts">
 import { defineComponent } from 'vue';
 import { api } from '@/http-api';
+import { ImageType } from '@/image';
 </script>
 <script lang="ts">
 export default defineComponent({
@@ -8,20 +9,35 @@ export default defineComponent({
 		id: {
 			type: Number,
 			required: true
+		},
+		images: {
+			type: Map<number, ImageType>,
+			required: true
+		},
+		redirect: {
+			type: String
+		}
+	},
+	data() {
+		return {
+			data: "" as string
 		}
 	},
 	methods: {
-		async getImage(id: number) {
+		getImage(id: number) {
+			if (this.images.has(id) && this.images.get(id)!.data != null) {
+				this.data = this.images.get(id)!.data;
+			} else {
+				this.downloadImage(id);
+			}
+		},
+		async downloadImage(id: number) {
 			const data = await api.getImage(id);
 			const reader = new window.FileReader();
 			reader.readAsDataURL(data);
 			reader.onload = () => {
-				const imgElt = document.getElementById("img-" + id) as HTMLImageElement;
-				if (imgElt !== null) {
-					if (reader.result as string) {
-						imgElt.setAttribute("src", (reader.result as string));
-					}
-				}
+				this.data = (reader.result as string)
+				this.images.get(id)!.data = this.data;
 			};
 		}
 	},
@@ -36,15 +52,24 @@ export default defineComponent({
 })
 </script>
 <template>
-	<div>
-		<a :href="'./images/' + id" target="_blank">
-			<img :id="'img-' + id" />
+	<div class="imageContainer">
+		<RouterLink v-if="redirect" :to="{ name: redirect, params: { id: id } }">
+			<img :id="'img-' + id" :src="data" />
+		</RouterLink>
+		<a v-else :href="'./images/' + id" target="_blank">
+			<img :id="'img-' + id" :src="data" />
 		</a>
 	</div>
 </template>
 
-<style scoped>
-img:hover {
+<style>
+.imageContainer img {
+	width: 100%;
+	height: 100%;
+	object-fit: contain;
+}
+
+.imageContainer:hover {
 	transform: scale(1.1);
 	transition: transform 0.5s;
 }