From 83e789e1d521f84049763e8fa7c1dd34131edc87 Mon Sep 17 00:00:00 2001
From: Cagatay Civici <cagatay.civici@gmail.com>
Date: Thu, 17 Sep 2020 14:58:41 +0300
Subject: [PATCH] Fixed lint errors and code styles

---
 src/AppMenu.vue                                | 12 ++++++------
 src/components/carousel/Carousel.vue           |  7 ++++---
 src/components/galleria/Galleria.vue           |  4 ++--
 src/components/galleria/GalleriaContent.vue    |  8 ++++----
 src/components/galleria/GalleriaItem.vue       |  4 ++--
 src/components/galleria/GalleriaThumbnails.vue |  2 +-
 src/router/index.js                            |  2 +-
 7 files changed, 20 insertions(+), 19 deletions(-)

diff --git a/src/AppMenu.vue b/src/AppMenu.vue
index 927d08a09..60b733448 100755
--- a/src/AppMenu.vue
+++ b/src/AppMenu.vue
@@ -70,7 +70,7 @@
 
             <div class="menu-category">Data</div>
             <div class="menu-items">
-                <router-link to="/datatable" v-slot="{ href, route, navigate, isActive }">
+                <router-link to="/datatable" v-slot="{isActive}">
                     <div>
                         <a tabindex="0" @click="toggleSubmenu($event, 'datatable')">DataTable</a>
                         <transition name="p-toggleable-content">
@@ -115,7 +115,7 @@
                 <router-link to="/organizationchart">OrganizationChart</router-link>
                 <router-link to="/paginator">Paginator</router-link>
                 <router-link to="/picklist">PickList</router-link>
-                <router-link to="/tree" v-slot="{ href, route, navigate, isActive }">
+                <router-link to="/tree" v-slot="{isActive}">
                     <div>
                         <a tabindex="0" @click="toggleSubmenu($event, 'tree')">Tree</a>
                         <transition name="p-toggleable-content">
@@ -131,7 +131,7 @@
                         </transition>
                     </div>
                 </router-link>
-                <router-link to="/treetable" v-slot="{ href, route, navigate, isActive }">
+                <router-link to="/treetable" v-slot="{isActive}">
                     <div>
                         <a tabindex="0" @click="toggleSubmenu($event, 'treetable')">TreeTable</a>
                         <transition name="p-toggleable-content">
@@ -215,7 +215,7 @@
             <div class="menu-category">Media</div>
             <div class="menu-items">
                 <router-link to="/carousel">Carousel</router-link>
-                <router-link to="/galleria" v-slot="{ href, route, navigate, isActive }">
+                <router-link to="/galleria" v-slot="{isActive}">
                     <div>
                         <a tabindex="0" @click="toggleSubmenu($event, 'galleria')">Galleria</a>
                         <transition name="p-toggleable-content">
@@ -270,8 +270,8 @@ export default {
             event.preventDefault();
         },
         isSubmenuActive(name, routerIsActive) {
-            if (this.activeSubmenus.hasOwnProperty(name)) {
-                return this.activeSubmenus[name];
+            if (this.activeSubmenus[name]) {
+                return true;
             }
             else if (routerIsActive) {
                 this.activeSubmenus[name] = true;
diff --git a/src/components/carousel/Carousel.vue b/src/components/carousel/Carousel.vue
index b5ba0110e..bf0943030 100755
--- a/src/components/carousel/Carousel.vue
+++ b/src/components/carousel/Carousel.vue
@@ -335,7 +335,8 @@ export default {
         `;
 
 			if (this.responsiveOptions) {
-				this.responsiveOptions.sort((data1, data2) => {
+				let _responsiveOptions = {...this.responsiveOptions};
+				_responsiveOptions.sort((data1, data2) => {
 					const value1 = data1.breakpoint;
 					const value2 = data2.breakpoint;
 					let result = null;
@@ -354,8 +355,8 @@ export default {
 					return -1 * result;
 				});
 
-				for (let i = 0; i < this.responsiveOptions.length; i++) {
-					let res = this.responsiveOptions[i];
+				for (let i = 0; i < _responsiveOptions.length; i++) {
+					let res = _responsiveOptions[i];
 
 					innerHTML += `
                     @media screen and (max-width: ${res.breakpoint}) {
diff --git a/src/components/galleria/Galleria.vue b/src/components/galleria/Galleria.vue
index 4ccf3c308..74b731e0e 100755
--- a/src/components/galleria/Galleria.vue
+++ b/src/components/galleria/Galleria.vue
@@ -1,11 +1,11 @@
 <template>
     <div v-if="fullScreen && (maskVisible || visible)" ref="mask" :class="maskContentClass">
         <transition name="p-galleria" @enter="onEnter" @before-leave="onBeforeLeave" @after-leave="onAfterLeave" @appear="onAppear">
-            <GalleriaContent v-if="visible" v-bind="$props" @maskHide="maskHide" :templates="$scopedSlots" @activeItemChange="onActiveItemChange" />
+            <GalleriaContent v-if="visible" v-bind="$props" @mask-hide="maskHide" :templates="$scopedSlots" @activeitem-change="onActiveItemChange" />
         </transition>
     </div>
 
-    <GalleriaContent v-else-if="!fullScreen" v-bind="$props" :templates="$scopedSlots" @activeItemChange="onActiveItemChange" />
+    <GalleriaContent v-else-if="!fullScreen" v-bind="$props" :templates="$scopedSlots" @activeitem-change="onActiveItemChange" />
 </template>
 
 <script>
diff --git a/src/components/galleria/GalleriaContent.vue b/src/components/galleria/GalleriaContent.vue
index a044b385f..4694b3abf 100755
--- a/src/components/galleria/GalleriaContent.vue
+++ b/src/components/galleria/GalleriaContent.vue
@@ -1,6 +1,6 @@
 <template>
     <div :id="id" v-if="$attrs.value && $attrs.value.length > 0" :class="galleriaClass" :style="$attrs.containerStyle">
-        <button v-if="$attrs.fullScreen" type="button" class="p-galleria-close p-link" @click="$emit('maskHide')" v-ripple>
+        <button v-if="$attrs.fullScreen" type="button" class="p-galleria-close p-link" @click="$emit('mask-hide')" v-ripple>
             <span class="p-galleria-close-icon pi pi-times"></span>
         </button>
         <div v-if="$attrs.templates && $attrs.templates['header']" class="p-galleria-header">
@@ -10,12 +10,12 @@
             <GalleriaItem :value="$attrs.value" :activeIndex.sync="activeIndex" :circular="$attrs.circular" :templates="$attrs.templates"
                 :showIndicators="$attrs.showIndicators" :changeItemOnIndicatorHover="$attrs.changeItemOnIndicatorHover"
                 :showItemNavigators="$attrs.showItemNavigators" :autoPlay="$attrs.autoPlay" :slideShowActive.sync="slideShowActive"
-                @startSlideShow="startSlideShow" @stopSlideShow="stopSlideShow" />
+                @start-slideshow="startSlideShow" @stop-slideshow="stopSlideShow" />
 
             <GalleriaThumbnails v-if="$attrs.showThumbnails" :containerId="id" :value="$attrs.value" :activeIndex.sync="activeIndex" :templates="$attrs.templates"
                 :numVisible="$attrs.numVisible" :responsiveOptions="$attrs.responsiveOptions" :circular="$attrs.circular"
                 :isVertical="isVertical()" :contentHeight="$attrs.verticalThumbnailViewPortHeight" :showThumbnailNavigators="$attrs.showThumbnailNavigators"
-                :slideShowActive.sync="slideShowActive" @stopSlideShow="stopSlideShow" />
+                :slideShowActive.sync="slideShowActive" @stop-slideshow="stopSlideShow" />
         </div>
         <div v-if="$attrs.templates && $attrs.templates['footer']" class="p-galleria-footer">
             <GalleriaItemSlot type="footer" :templates="$attrs.templates"/>
@@ -46,7 +46,7 @@ export default {
         }
     },
     updated() {
-        this.$emit('activeItemChange', this.activeIndex);
+        this.$emit('activeitem-change', this.activeIndex);
     },
     beforeDestroy() {
 		if (this.slideShowActive) {
diff --git a/src/components/galleria/GalleriaItem.vue b/src/components/galleria/GalleriaItem.vue
index f1cfa6cec..280e3ec89 100755
--- a/src/components/galleria/GalleriaItem.vue
+++ b/src/components/galleria/GalleriaItem.vue
@@ -70,7 +70,7 @@ export default {
     },
     mounted() {
         if (this.autoPlay) {
-            this.$emit('startSlideShow');
+            this.$emit('start-slideshow');
         }
     },
     methods: {
@@ -92,7 +92,7 @@ export default {
         },
         stopSlideShow() {
             if (this.slideShowActive && this.stopSlideShow) {
-                this.$emit('stopSlideShow');
+                this.$emit('stop-slideshow');
             }
         },
         navBackward(e) {
diff --git a/src/components/galleria/GalleriaThumbnails.vue b/src/components/galleria/GalleriaThumbnails.vue
index f72332bd5..843e05394 100755
--- a/src/components/galleria/GalleriaThumbnails.vue
+++ b/src/components/galleria/GalleriaThumbnails.vue
@@ -179,7 +179,7 @@ export default {
         },
         stopSlideShow() {
             if (this.slideShowActive && this.stopSlideShow) {
-                this.$emit('stopSlideShow');
+                this.$emit('stop-slideshow');
             }
         },
         getMedianItemIndex() {
diff --git a/src/router/index.js b/src/router/index.js
index 607b6c0cc..a312c0a25 100644
--- a/src/router/index.js
+++ b/src/router/index.js
@@ -737,5 +737,5 @@ const router = createRouter({
     history: createWebHashHistory(),
     routes
 });
-  
+
 export default router;
\ No newline at end of file