From 1897c40dde1826a9120bdf68687f9f548ea3b77a Mon Sep 17 00:00:00 2001
From: Anya Wallace <aswallace98@gmail.com>
Date: Wed, 29 Mar 2023 16:55:03 -0700
Subject: [PATCH] fix(#373): adjust icon toolbar and fix linting

---
 app/src/assets/css/modules/_utility.scss      | 24 ++++++++++++-------
 app/src/modules/vega-chart.js                 |  2 +-
 app/src/pages/explorer/Gallery.vue            | 10 ++++----
 .../explorer/chart/view/vega-view-script.js   |  8 +++----
 4 files changed, 26 insertions(+), 18 deletions(-)

diff --git a/app/src/assets/css/modules/_utility.scss b/app/src/assets/css/modules/_utility.scss
index e13d08d3..63b37b0d 100644
--- a/app/src/assets/css/modules/_utility.scss
+++ b/app/src/assets/css/modules/_utility.scss
@@ -593,20 +593,28 @@
     &-gridicon {
         position: absolute;
         right: 4.5rem;
+        top: 16rem;
         z-index: 10000000;
         @include respond(tab-land){
             right: 2.5rem;
+            top: 19rem;
+        }
+        @include respond(phone) {
+            top: 15rem;
         }
         & > div {
             display: inline-block;
-
-            & > i {
-                font-size: 1rem !important;
-                width: 21px !important;
-                min-width: 21px !important;
-                height: 21px !important; 
-                cursor: pointer;
-            }
+        }
+        .md-icon-button {
+            width: 3rem !important;
+            min-width: 3rem !important
+        }
+        .md-icon {
+            font-size: 2.5rem !important;
+            width: 2.5rem !important;
+            min-width: 2.5rem !important;
+            height: 2.5rem !important; 
+            cursor: pointer;
         }
 
         &_explorer {
diff --git a/app/src/modules/vega-chart.js b/app/src/modules/vega-chart.js
index 48540a31..afe83775 100644
--- a/app/src/modules/vega-chart.js
+++ b/app/src/modules/vega-chart.js
@@ -149,7 +149,7 @@ async function loadChart (chartUri) {
     // lodPrefix is now fixed (e.g. http://nanomine.org/viz/chartId instead of http://purl.org/chart/view/chartId)
     // Todo (ticket-xx): Remove this if logic if response is consistent
     chartUrl = decodeURIComponent(chartUri.split('view/')[1])
-  } else if ( !chartUri.startsWith('http') ) {
+  } else if (!chartUri.startsWith('http')) {
     // This is currently needed for editing since gallery only passes ID and not the full uri
     chartUrl = `${chartUriPrefix}${chartUri}`
   }
diff --git a/app/src/pages/explorer/Gallery.vue b/app/src/pages/explorer/Gallery.vue
index de3ceaa1..ad49f2a2 100644
--- a/app/src/pages/explorer/Gallery.vue
+++ b/app/src/pages/explorer/Gallery.vue
@@ -93,16 +93,16 @@
             with identifier <b>{{dialog.chart.identifier}}</b>.
           </md-content>
         </div>
-        <div v-if="dialogLoading">      
+        <div v-if="dialogLoading">
           <spinner
             :loading="dialogLoading"
             text='Deleting Chart'
           />
         </div>
-      </template> 
+      </template>
       <template v-slot:actions>
         <span v-if="dialog.type=='delete' && dialog.chart">
-          <md-button @click.native.prevent="toggleDialogBox"> 
+          <md-button @click.native.prevent="toggleDialogBox">
             No, cancel
           </md-button>
           <md-button @click.native.prevent="deleteChart(dialog.chart)">
@@ -172,8 +172,8 @@ export default {
     async deleteChart (chart) {
       if (!this.isAdmin) return // temporary safeguard
       this.dialogLoading = true
-      await this.$store.dispatch('explorer/curation/deleteChartNanopub', chart.identifier )
-      await this.$store.dispatch('explorer/curation/deleteChartES', chart.identifier )
+      await this.$store.dispatch('explorer/curation/deleteChartNanopub', chart.identifier)
+      await this.$store.dispatch('explorer/curation/deleteChartES', chart.identifier)
       this.toggleDialogBox()
       this.dialogLoading = false
       await this.loadItems()
diff --git a/app/src/pages/explorer/chart/view/vega-view-script.js b/app/src/pages/explorer/chart/view/vega-view-script.js
index d7861eb3..7b165b17 100644
--- a/app/src/pages/explorer/chart/view/vega-view-script.js
+++ b/app/src/pages/explorer/chart/view/vega-view-script.js
@@ -52,7 +52,7 @@ export default {
     ...mapGetters({
       dialogBoxActive: 'dialogBox',
       isAuth: 'auth/isAuthenticated',
-      isAdmin: 'auth/isAdmin',
+      isAdmin: 'auth/isAdmin'
     }),
     specViewerSpec () {
       return this.specViewer.includeData ? this.spec : this.chart && this.chart.baseSpec
@@ -87,11 +87,11 @@ export default {
     editChart () {
       return this.$router.push(`/explorer/chart/editor/edit/${this.chartId}`)
     },
-    async deleteChart() {
+    async deleteChart () {
       if (!this.isAdmin) return // temporary safeguard
       this.dialogLoading = true
-      await this.$store.dispatch('explorer/curation/deleteChartNanopub', this.chart.uri )
-      await this.$store.dispatch('explorer/curation/deleteChartES', this.chart.uri )
+      await this.$store.dispatch('explorer/curation/deleteChartNanopub', this.chart.uri)
+      await this.$store.dispatch('explorer/curation/deleteChartES', this.chart.uri)
       this.toggleDialogBox()
       this.dialogLoading = false
       this.$router.push('/explorer/chart')