From 33e5f00432f085221ffc9a3ab4a2471e5fa27e5b Mon Sep 17 00:00:00 2001
From: rijkvanzanten <rijkvanzanten@me.com>
Date: Fri, 11 Jun 2021 21:50:13 -0400
Subject: [PATCH] Fix buttons on panels

---
 app/src/modules/insights/components/panel.vue         |  4 +++-
 app/src/modules/insights/routes/dashboard.vue         | 11 ++---------
 .../modules/insights/routes/panel-configuration.vue   |  1 +
 3 files changed, 6 insertions(+), 10 deletions(-)

diff --git a/app/src/modules/insights/components/panel.vue b/app/src/modules/insights/components/panel.vue
index d402bc9c4d..7d3dcdb5c7 100644
--- a/app/src/modules/insights/components/panel.vue
+++ b/app/src/modules/insights/components/panel.vue
@@ -26,14 +26,16 @@
 				name="control_point_duplicate"
 				v-tooltip="t('duplicate')"
 				@click.stop="$emit('duplicate')"
+				clickable
 			/>
 			<v-icon
 				class="edit-icon"
 				name="edit"
 				v-tooltip="t('edit')"
 				@click.stop="$router.push(`/insights/${panel.dashboard}/${panel.id}`)"
+				clickable
 			/>
-			<v-icon class="delete-icon" name="clear" v-tooltip="t('delete')" @click.stop="$emit('delete')" />
+			<v-icon clickable class="delete-icon" name="clear" v-tooltip="t('delete')" @click.stop="$emit('delete')" />
 		</div>
 
 		<div class="resize-details">
diff --git a/app/src/modules/insights/routes/dashboard.vue b/app/src/modules/insights/routes/dashboard.vue
index 3b0db78674..14a8456ae3 100644
--- a/app/src/modules/insights/routes/dashboard.vue
+++ b/app/src/modules/insights/routes/dashboard.vue
@@ -67,7 +67,7 @@
 			@cancel="$router.push(`/insights/${primaryKey}`)"
 		/>
 
-		<v-dialog :active="!!confirmDeletePanel" @esc="confirmDeletePanel = null">
+		<v-dialog :model-value="!!confirmDeletePanel" @esc="confirmDeletePanel = null">
 			<v-card>
 				<v-card-title>{{ t('panel_delete_confirm') }}</v-card-title>
 
@@ -75,7 +75,7 @@
 					<v-button @click="confirmDeletePanel = null" secondary>
 						{{ t('cancel') }}
 					</v-button>
-					<v-button class="action-delete" @click="deletePanel" :loading="deletingPanel">
+					<v-button danger @click="deletePanel" :loading="deletingPanel">
 						{{ t('delete') }}
 					</v-button>
 				</v-card-actions>
@@ -380,13 +380,6 @@ export default defineComponent({
 	opacity: 1;
 }
 
-.action-delete {
-	--v-button-background-color: var(--danger-10);
-	--v-button-color: var(--danger);
-	--v-button-background-color-hover: var(--danger-25);
-	--v-button-color-hover: var(--danger);
-}
-
 .fullscreen,
 .clear-changes {
 	--v-button-color: var(--foreground-normal);
diff --git a/app/src/modules/insights/routes/panel-configuration.vue b/app/src/modules/insights/routes/panel-configuration.vue
index b63067380b..3685a8f944 100644
--- a/app/src/modules/insights/routes/panel-configuration.vue
+++ b/app/src/modules/insights/routes/panel-configuration.vue
@@ -4,6 +4,7 @@
 		:title="(panel && panel.name) || t('panel')"
 		@cancel="$emit('cancel')"
 		icon="insert_chart"
+		persistent
 	>
 		<template #actions>
 			<v-button :disabled="!edits.type" @click="emitSave" icon rounded v-tooltip.bottom="t('done')">