From 40cfed4240be385653f2c2a680a042d36f28e951 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Julius=20H=C3=A4rtl?= <jus@bitgrid.net>
Date: Wed, 4 Nov 2020 18:30:03 +0100
Subject: [PATCH] Move to icon buttons
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Julius Härtl <jus@bitgrid.net>
---
 .../navigation/AppNavigationBoard.vue         | 35 +++++++++++--------
 src/store/main.js                             | 16 ++++++++-
 2 files changed, 36 insertions(+), 15 deletions(-)

diff --git a/src/components/navigation/AppNavigationBoard.vue b/src/components/navigation/AppNavigationBoard.vue
index 03647959a..3c6a5458a 100644
--- a/src/components/navigation/AppNavigationBoard.vue
+++ b/src/components/navigation/AppNavigationBoard.vue
@@ -68,24 +68,28 @@
 			<ActionButton :icon="!showDueSettings ? 'icon-notifications-dark' : 'icon-view-previous' " @click="showDueSettings=!showDueSettings">
 				{{ t('deck', 'Due date reminders') }}
 			</ActionButton>
-			<ActionRadio v-if="showDueSettings"
+
+			<ActionButton v-if="showDueSettings"
 				name="notification"
-				:checked="board.settings['notify-due'] === 'all'"
-				@change="updateSetting('notify-due', 'all')">
+				icon="icon-sound"
+				:class="{ 'forced-active': board.settings['notify-due'] === 'all' }"
+				@click="updateSetting('notify-due', 'all')">
 				{{ t('deck', 'All cards') }}
-			</ActionRadio>
-			<ActionRadio v-if="showDueSettings"
+			</ActionButton>
+			<ActionButton v-if="showDueSettings"
 				name="notification"
-				:checked="board.settings['notify-due'] === 'assigned'"
-				@change="updateSetting('notify-due', 'assigned')">
+				icon="icon-user"
+				:class="{ 'forced-active': board.settings['notify-due'] === 'assigned' }"
+				@click="updateSetting('notify-due', 'assigned')">
 				{{ t('deck', 'Assigned cards') }}
-			</ActionRadio>
-			<ActionRadio v-if="showDueSettings"
+			</ActionButton>
+			<ActionButton v-if="showDueSettings"
 				name="notification"
-				:checked="board.settings['notify-due'] === 'off'"
-				@change="updateSetting('notify-due', 'off')">
+				icon="icon-sound-off"
+				:class="{ 'forced-active': board.settings['notify-due'] === 'off' }"
+				@click="updateSetting('notify-due', 'off')">
 				{{ t('deck', 'No notifications') }}
-			</ActionRadio>
+			</ActionButton>
 
 			<ActionButton v-if="canManage && !showDueSettings"
 				icon="icon-delete"
@@ -108,7 +112,7 @@
 </template>
 
 <script>
-import { AppNavigationIconBullet, AppNavigationCounter, AppNavigationItem, ColorPicker, Actions, ActionButton, ActionRadio } from '@nextcloud/vue'
+import { AppNavigationIconBullet, AppNavigationCounter, AppNavigationItem, ColorPicker, Actions, ActionButton } from '@nextcloud/vue'
 import ClickOutside from 'vue-click-outside'
 
 export default {
@@ -120,7 +124,6 @@ export default {
 		ColorPicker,
 		Actions,
 		ActionButton,
-		ActionRadio,
 	},
 	directives: {
 		ClickOutside,
@@ -299,4 +302,8 @@ export default {
 			background-size: 14px;
 		}
 	}
+
+	.forced-active {
+		box-shadow: inset 4px 0 var(--color-primary-element);
+	}
 </style>
diff --git a/src/store/main.js b/src/store/main.js
index 1acec6466..d6ee6f5e7 100644
--- a/src/store/main.js
+++ b/src/store/main.js
@@ -138,7 +138,21 @@ export default new Vuex.Store({
 	},
 	mutations: {
 		SET_CONFIG(state, { key, value }) {
-			Vue.set(state.config, key, value)
+			const [scope, id, configKey] = key.split(':', 3)
+			let indexExisting = -1
+			switch (scope) {
+			case 'board':
+				indexExisting = state.boards.findIndex((b) => {
+					return id === '' + b.id
+				})
+
+				if (indexExisting > -1) {
+					Vue.set(state.boards[indexExisting].settings, configKey, value)
+				}
+				break
+			default:
+				Vue.set(state.config, key, value)
+			}
 		},
 		setSearchQuery(state, searchQuery) {
 			state.searchQuery = searchQuery