From 98688b81c035b45c17e601b5e258d7510a066b54 Mon Sep 17 00:00:00 2001
From: Peter Papp <peterpapp@makingcg.com>
Date: Tue, 10 Aug 2021 11:43:05 +0200
Subject: [PATCH] file preview refactoring

---
 .../FilePreview.vue                           |   4 +-
 .../FilePreviewMedia.vue                      | 175 ++----------------
 .../FilePreviewToolbar.vue                    |   0
 .../js/components/FilePreview/Media/Audio.vue |  17 ++
 .../FilePreview/Media/ImageFile.vue           |  17 ++
 .../components/FilePreview/Media/PdfFile.vue  | 114 ++++++++++++
 .../js/components/FilePreview/Media/Video.vue |  60 ++++++
 resources/js/views/Platform.vue               |   2 +-
 resources/js/views/Shared.vue                 |   2 +-
 9 files changed, 228 insertions(+), 163 deletions(-)
 rename resources/js/components/{FilesView => FilePreview}/FilePreview.vue (90%)
 rename resources/js/components/{FilesView => FilePreview}/FilePreviewMedia.vue (55%)
 rename resources/js/components/{FilesView => FilePreview}/FilePreviewToolbar.vue (100%)
 create mode 100644 resources/js/components/FilePreview/Media/Audio.vue
 create mode 100644 resources/js/components/FilePreview/Media/ImageFile.vue
 create mode 100644 resources/js/components/FilePreview/Media/PdfFile.vue
 create mode 100644 resources/js/components/FilePreview/Media/Video.vue

diff --git a/resources/js/components/FilesView/FilePreview.vue b/resources/js/components/FilePreview/FilePreview.vue
similarity index 90%
rename from resources/js/components/FilesView/FilePreview.vue
rename to resources/js/components/FilePreview/FilePreview.vue
index 895401ce..7ddc2971 100644
--- a/resources/js/components/FilesView/FilePreview.vue
+++ b/resources/js/components/FilePreview/FilePreview.vue
@@ -14,8 +14,8 @@
 </template>
 
 <script>
-    import FilePreviewToolbar from '@/components/FilesView/FilePreviewToolbar'
-    import FilePreviewMedia from '@/components/FilesView/FilePreviewMedia'
+    import FilePreviewToolbar from '@/components/FilePreview/FilePreviewToolbar'
+    import FilePreviewMedia from '@/components/FilePreview/FilePreviewMedia'
     import {events} from '@/bus'
 
     export default {
diff --git a/resources/js/components/FilesView/FilePreviewMedia.vue b/resources/js/components/FilePreview/FilePreviewMedia.vue
similarity index 55%
rename from resources/js/components/FilesView/FilePreviewMedia.vue
rename to resources/js/components/FilePreview/FilePreviewMedia.vue
index fa2888f8..d46256d5 100644
--- a/resources/js/components/FilesView/FilePreviewMedia.vue
+++ b/resources/js/components/FilePreview/FilePreviewMedia.vue
@@ -1,5 +1,5 @@
 <template>
-	<div class="media-full-preview" id="mediaPreview" v-if="currentFile">
+	<div v-if="currentFile" class="file-preview-wrapper">
 
         <!--Arrow navigation-->
         <div v-if="files.length > 1" class="navigation-arrows">
@@ -16,45 +16,13 @@
 		<div class="file-wrapper-preview">
 
             <!--Show PDF-->
-            <div v-if="isPDF" id="pdf-wrapper" :style="{width: documentSize + '%'}">
-                <pdf :src="pdfData" v-for="i in numPages" :key="i" :resize="true" :page="i" scale="page-width" style="width:100%; margin:0 auto 35px;" id="printable-file" class="pdf-file">
-                    <template slot="loading">
-                        <h1>loading content...</h1>
-                    </template>
-                </pdf>
-            </div>
+			<PdfFile v-if="isPDF" :file="currentFile"/>
 
             <!--Show Audio, Video and Image-->
             <div v-if="isAudio || isImage || isVideo" class="file-wrapper">
-
-				<audio
-                    v-if="isAudio"
-                    :class="{'file-shadow': ! $isMobile() }"
-                    class="file audio"
-                    :src="currentFile.file_url"
-                    controls>
-                </audio>
-
-                <img
-                    id="printable-file"
-                    v-if="isImage"
-                    class="file"
-                    :class="{'file-shadow': !$isMobile() }"
-                    :src="currentFile.file_url"
-                />
-
-                <div class="video-wrapper" v-if="isVideo">
-					<video
-                        :src="currentFile.file_url"
-                        class="video"
-                        :class="{'file-shadow': !$isMobile() }"
-                        controlsList="nodownload"
-                        disablePictureInPicture
-                        playsinline
-                        controls
-                        autoplay
-                    />
-				</div>
+				<Audio v-if="isAudio" :file="currentFile"/>
+				<Video v-if="isVideo" :file="currentFile"/>
+				<ImageFile v-if="isImage" :file="currentFile"/>
 			</div>
 		</div>
 	</div>
@@ -63,10 +31,13 @@
 <script>
 import {ChevronLeftIcon, ChevronRightIcon} from 'vue-feather-icons'
 import ToolbarButton from '@/components/FilesView/ToolbarButton'
+import ImageFile from '@/components/FilePreview/Media/ImageFile'
+import PdfFile from '@/components/FilePreview/Media/PdfFile'
+import Audio from '@/components/FilePreview/Media/Audio'
+import Video from '@/components/FilePreview/Media/Video'
 import Spinner from '@/components/FilesView/Spinner'
 import {mapGetters} from 'vuex'
 import {events} from '@/bus'
-import pdf from 'pdfvuer'
 
 export default {
     name: 'FilePreviewMedia',
@@ -74,8 +45,11 @@ export default {
         ChevronRightIcon,
         ChevronLeftIcon,
         ToolbarButton,
+		ImageFile,
+		PdfFile,
         Spinner,
-        pdf,
+		Audio,
+		Video,
     },
     computed: {
         ...mapGetters([
@@ -84,12 +58,6 @@ export default {
             'entries',
         ]),
         currentFile() {
-
-        	console.log(this.fastPreview);
-        	console.log(this.files[Math.abs(this.currentIndex) % this.files.length]);
-        	console.log(this.currentIndex);
-        	console.log(this.files);
-
             return this.fastPreview
 				? this.fastPreview
 				: this.files[Math.abs(this.currentIndex) % this.files.length]
@@ -109,27 +77,19 @@ export default {
     },
     data() {
         return {
-            pdfData: undefined,
-            numPages: 0,
             currentIndex: 0,
             files: [],
-            documentSize: 50,
         }
     },
     watch: {
         files() {
             if (this.files.length === 0)
-                events.$emit('file-preview:hide')
+                events.$emit('file-preview-wrapper:hide')
         },
         currentFile() {
             if (this.clipboard[0]) {
                 this.$store.commit('CLIPBOARD_CLEAR')
                 this.$store.commit('ADD_ITEM_TO_CLIPBOARD', this.currentFile)
-
-                // Init pdf instance
-                if (this.currentFile.mimetype === 'pdf') {
-                    this.getPdf()
-                }
             }
         },
         clipboard() {
@@ -148,16 +108,6 @@ export default {
         },
     },
     methods: {
-        getPdf() {
-            this.pdfData = undefined
-            this.numPages = 0
-
-            let self = this;
-
-            self.pdfData = pdf.createLoadingTask(this.currentFile.file_url);
-
-            self.pdfData.then(pdf => self.numPages = pdf.numPages);
-        },
         getFilesForView() {
             let requestedFile = this.clipboard[0]
 
@@ -181,22 +131,9 @@ export default {
                 }
             })
         },
-		getDocumentSize() {
-			if (window.innerWidth < 960) {
-				this.documentSize = 100
-			}
-
-			if (window.innerWidth > 960){
-				this.documentSize = localStorage.getItem('documentSize')
-					? parseInt(localStorage.getItem('documentSize'))
-					: 50;
-			}
-		},
 		next() {
 			if (!this.files.length > 1) return
 
-			this.pdfData = undefined
-
 			this.currentIndex += 1
 
 			if (this.currentIndex > this.files.length - 1) {
@@ -206,40 +143,22 @@ export default {
 		prev() {
 			if (!this.files.length > 1) return
 
-			this.pdfData = undefined
-
 			this.currentIndex -= 1
 
 			if (this.currentIndex < 0) {
 				this.currentIndex = this.files.length - 1
 			}
-		},
-		zoomIn() {
-			if (this.documentSize < 100) {
-				this.documentSize += 10
-				localStorage.setItem('documentSize', this.documentSize)
-			}
-		},
-		zoomOut() {
-			if (this.documentSize > 40) {
-				this.documentSize -= 10
-				localStorage.setItem('documentSize', this.documentSize)
-			}
 		}
     },
     created() {
 		events.$on('file-preview:next', () => this.next())
         events.$on('file-preview:prev', () => this.prev())
-        events.$on('document-zoom:in', () => this.zoomIn())
-        events.$on('document-zoom:out', () => this.zoomOut())
 
-        this.getDocumentSize()
         this.getFilesForView()
     }
 }
 </script>
 
-<style src="pdfvuer/dist/pdfvuer.css" lang="css"></style>
 <style lang="scss">
 @import '@assets/vuefilemanager/_variables';
 @import '@assets/vuefilemanager/_mixins';
@@ -270,26 +189,7 @@ export default {
     }
 }
 
-#pdf-wrapper {
-	border-radius: 8px;
-	overflow-y: scroll;
-	margin: 0 auto;
-	position: absolute;
-	top: 0;
-	left: 0;
-	right: 0;
-	bottom: 0;
-	z-index: 1;
-	padding: 40px;
-
-	.pdf-file {
-		box-shadow: $light_mode_popup_shadow;
-		border-radius: 8px;
-		overflow: hidden;
-	}
-}
-
-.media-full-preview {
+.file-preview-wrapper {
     height: calc(100% - 72px);
     top: 72px;
     position: relative;
@@ -330,57 +230,14 @@ export default {
         img {
             border-radius: 4px;
         }
-
-        .video-wrapper {
-            max-width: 1080px;
-            max-height: 100%;
-
-            @media (min-width: 1200px) {
-                & {
-                    max-width: 800px;
-                }
-            }
-
-            @media (min-width: 1920px) and (max-width: 2560px) {
-                & {
-                    max-width: 1080px;
-                }
-            }
-            @media (min-width: 2560px) and (max-width: 3840px) {
-                & {
-                    max-width: 1440px;
-                }
-            }
-            @media (min-width: 3840px) {
-                & {
-                    max-width: 2160px;
-                }
-            }
-
-            .video {
-                max-width: 100%;
-                max-height: 100%;
-                align-self: center;
-            }
-        }
     }
 }
 
 @media only screen and (max-width: 960px) {
 
-    .media-full-preview {
+    .file-preview-wrapper {
         top: 53px;
     }
-
-	#pdf-wrapper {
-		border-radius: 0;
-		padding: 0;
-
-		.pdf-file {
-			box-shadow: none;
-			border-radius: 0;
-		}
-	}
 }
 
 
diff --git a/resources/js/components/FilesView/FilePreviewToolbar.vue b/resources/js/components/FilePreview/FilePreviewToolbar.vue
similarity index 100%
rename from resources/js/components/FilesView/FilePreviewToolbar.vue
rename to resources/js/components/FilePreview/FilePreviewToolbar.vue
diff --git a/resources/js/components/FilePreview/Media/Audio.vue b/resources/js/components/FilePreview/Media/Audio.vue
new file mode 100644
index 00000000..342c0ed7
--- /dev/null
+++ b/resources/js/components/FilePreview/Media/Audio.vue
@@ -0,0 +1,17 @@
+<template>
+	<audio
+		:class="{'file-shadow': ! this.$isMobile() }"
+		class="file audio"
+		:src="file.file_url"
+		controls>
+	</audio>
+</template>
+
+<script>
+	export default {
+		name: 'Audio',
+		props: [
+			'file'
+		],
+	}
+</script>
\ No newline at end of file
diff --git a/resources/js/components/FilePreview/Media/ImageFile.vue b/resources/js/components/FilePreview/Media/ImageFile.vue
new file mode 100644
index 00000000..7dff7cca
--- /dev/null
+++ b/resources/js/components/FilePreview/Media/ImageFile.vue
@@ -0,0 +1,17 @@
+<template>
+	<img
+		id="printable-file"
+		class="file"
+		:class="{'file-shadow': ! this.$isMobile() }"
+		:src="file.file_url"
+	/>
+</template>
+
+<script>
+	export default {
+		name: 'ImageFile',
+		props: [
+			'file'
+		],
+	}
+</script>
\ No newline at end of file
diff --git a/resources/js/components/FilePreview/Media/PdfFile.vue b/resources/js/components/FilePreview/Media/PdfFile.vue
new file mode 100644
index 00000000..ed27fd88
--- /dev/null
+++ b/resources/js/components/FilePreview/Media/PdfFile.vue
@@ -0,0 +1,114 @@
+<template>
+	<div id="pdf-wrapper" :style="{width: documentSize + '%'}">
+		<pdf :src="pdfData" v-for="i in numPages" :key="i" :resize="true" :page="i" scale="page-width" style="width:100%; margin:0 auto 35px;" id="printable-file" class="pdf-file">
+			<template slot="loading">
+				<h1>loading content...</h1>
+			</template>
+		</pdf>
+	</div>
+</template>
+
+<script>
+	import {events} from '@/bus'
+	import pdf from 'pdfvuer'
+
+	export default {
+		name: 'PdfFile',
+		components: {
+			pdf,
+		},
+		props: [
+			'file'
+		],
+		watch: {
+			file() {
+				this.getPdf()
+			}
+		},
+		data() {
+			return {
+				pdfData: undefined,
+				numPages: 0,
+				documentSize: 50,
+			}
+		},
+		methods: {
+			getPdf() {
+				this.pdfData = undefined
+				this.numPages = 0
+
+				let self = this;
+
+				self.pdfData = pdf.createLoadingTask(this.file.file_url);
+
+				self.pdfData.then(pdf => self.numPages = pdf.numPages);
+			},
+			getDocumentSize() {
+				if (window.innerWidth < 960) {
+					this.documentSize = 100
+				}
+
+				if (window.innerWidth > 960){
+					this.documentSize = localStorage.getItem('documentSize')
+						? parseInt(localStorage.getItem('documentSize'))
+						: 50;
+				}
+			},
+			zoomIn() {
+				if (this.documentSize < 100) {
+					this.documentSize += 10
+					localStorage.setItem('documentSize', this.documentSize)
+				}
+			},
+			zoomOut() {
+				if (this.documentSize > 40) {
+					this.documentSize -= 10
+					localStorage.setItem('documentSize', this.documentSize)
+				}
+			}
+		},
+		created() {
+			this.getDocumentSize()
+			this.getPdf()
+
+			events.$on('document-zoom:in', () => this.zoomIn())
+			events.$on('document-zoom:out', () => this.zoomOut())
+		}
+	}
+</script>
+
+<style src="pdfvuer/dist/pdfvuer.css" lang="css"></style>
+<style lang="scss" scoped>
+    @import '@assets/vuefilemanager/_variables';
+
+	#pdf-wrapper {
+		border-radius: 8px;
+		overflow-y: scroll;
+		margin: 0 auto;
+		position: absolute;
+		top: 0;
+		left: 0;
+		right: 0;
+		bottom: 0;
+		z-index: 1;
+		padding: 40px;
+
+		.pdf-file {
+			box-shadow: $light_mode_popup_shadow;
+			border-radius: 8px;
+			overflow: hidden;
+		}
+	}
+
+	@media only screen and (max-width: 960px) {
+		#pdf-wrapper {
+			border-radius: 0;
+			padding: 0;
+
+			.pdf-file {
+				box-shadow: none;
+				border-radius: 0;
+			}
+		}
+	}
+</style>
\ No newline at end of file
diff --git a/resources/js/components/FilePreview/Media/Video.vue b/resources/js/components/FilePreview/Media/Video.vue
new file mode 100644
index 00000000..73544131
--- /dev/null
+++ b/resources/js/components/FilePreview/Media/Video.vue
@@ -0,0 +1,60 @@
+<template>
+	<div class="video-wrapper">
+		<video
+			:src="file.file_url"
+			class="video"
+			:class="{'file-shadow': !this.$isMobile() }"
+			controlsList="nodownload"
+			disablePictureInPicture
+			playsinline
+			controls
+			autoplay
+		/>
+	</div>
+</template>
+
+<script>
+	export default {
+		name: 'Video',
+		props: [
+			'file'
+		],
+	}
+</script>
+
+<style lang="scss" scoped>
+    @import '@assets/vuefilemanager/_variables';
+
+	.video-wrapper {
+		max-width: 1080px;
+		max-height: 100%;
+
+		@media (min-width: 1200px) {
+			& {
+				max-width: 800px;
+			}
+		}
+
+		@media (min-width: 1920px) and (max-width: 2560px) {
+			& {
+				max-width: 1080px;
+			}
+		}
+		@media (min-width: 2560px) and (max-width: 3840px) {
+			& {
+				max-width: 1440px;
+			}
+		}
+		@media (min-width: 3840px) {
+			& {
+				max-width: 2160px;
+			}
+		}
+
+		.video {
+			max-width: 100%;
+			max-height: 100%;
+			align-self: center;
+		}
+	}
+</style>
\ No newline at end of file
diff --git a/resources/js/views/Platform.vue b/resources/js/views/Platform.vue
index 9669d1ac..76d338d3 100644
--- a/resources/js/views/Platform.vue
+++ b/resources/js/views/Platform.vue
@@ -54,7 +54,7 @@
     import RenameItemPopup from '@/components/Others/RenameItemPopup'
     import ShareEditPopup from '@/components/Others/ShareEditPopup'
     import MoveItemPopup from '@/components/Others/MoveItemPopup'
-    import FilePreview from '@/components/FilesView/FilePreview'
+    import FilePreview from '@/components/FilePreview/FilePreview'
     import DragUI from '@/components/FilesView/DragUI'
     import {mapGetters} from 'vuex'
     import {events} from '@/bus'
diff --git a/resources/js/views/Shared.vue b/resources/js/views/Shared.vue
index 67a0c5be..11a35cc1 100644
--- a/resources/js/views/Shared.vue
+++ b/resources/js/views/Shared.vue
@@ -38,7 +38,7 @@
     import FileMenuMobile from '@/components/FilesView/FileMenuMobile'
     import RenameItemPopup from '@/components/Others/RenameItemPopup'
     import MoveItemPopup from '@/components/Others/MoveItemPopup'
-    import FilePreview from '@/components/FilesView/FilePreview'
+    import FilePreview from '@/components/FilePreview/FilePreview'
     import Spinner from '@/components/FilesView/Spinner'
     import Vignette from '@/components/Others/Vignette'
     import DragUI from '@/components/FilesView/DragUI'