From 5ae2882334ad9b34f53a6b30e5a4438e26eed6e3 Mon Sep 17 00:00:00 2001
From: mertmit <mertmit99@gmail.com>
Date: Sat, 3 Sep 2022 18:13:35 +0300
Subject: [PATCH] fix: use events instead of watch for color handling

Signed-off-by: mertmit <mertmit99@gmail.com>
---
 .../components/general/ColorPicker.vue        | 22 +++---
 packages/nc-gui-v2/composables/useProject.ts  | 16 +++--
 .../pages/[projectType]/[projectId]/index.vue | 70 +++++++++----------
 .../nc-gui-v2/pages/index/index/index.vue     | 67 ++++++------------
 4 files changed, 77 insertions(+), 98 deletions(-)

diff --git a/packages/nc-gui-v2/components/general/ColorPicker.vue b/packages/nc-gui-v2/components/general/ColorPicker.vue
index e5bb3bdc1d..5302c398ab 100644
--- a/packages/nc-gui-v2/components/general/ColorPicker.vue
+++ b/packages/nc-gui-v2/components/general/ColorPicker.vue
@@ -1,5 +1,4 @@
 <script lang="ts" setup>
-import { Chrome } from '@ckpack/vue-color'
 import { computed, enumColor, ref, watch } from '#imports'
 
 interface Props {
@@ -18,27 +17,28 @@ const props = withDefaults(defineProps<Props>(), {
   pickButton: false,
 })
 
-const emit = defineEmits(['update:modelValue'])
+const emit = defineEmits(['update:modelValue', 'input'])
 
 const vModel = computed({
   get: () => props.modelValue,
   set: (val) => {
-    emit('update:modelValue', val.hex8 ? val.hex8 : val || null)
+    emit('update:modelValue', val || null)
+    emit('input', val || null)
   },
 })
 
-const picked = ref<string | Record<string, any>>(props.modelValue || enumColor.light[0])
+const picked = ref<string>(props.modelValue || enumColor.light[0])
 
-const selectColor = (color: string | Record<string, any>) => {
-  picked.value = typeof color === 'string' ? color : color.hex8 ? color.hex8 : color
-  vModel.value = typeof color === 'string' ? color : color.hex8 ? color.hex8 : color
+const selectColor = (color: string) => {
+  picked.value = color
+  if (props.pickButton) vModel.value = color
 }
 
 const compare = (colorA: string, colorB: string) => colorA.toLowerCase() === colorB.toLowerCase()
 
 watch(picked, (n, _o) => {
   if (!props.pickButton) {
-    vModel.value = typeof n === 'string' ? n : n.hex8 ? n.hex8 : n
+    vModel.value = n
   }
 })
 </script>
@@ -50,11 +50,11 @@ watch(picked, (n, _o) => {
         v-for="(color, i) of colors.slice((colId - 1) * rowSize, colId * rowSize)"
         :key="`color-${colId}-${i}`"
         class="color-selector"
-        :class="compare(typeof picked === 'string' ? picked : picked.hex8, color) ? 'selected' : ''"
+        :class="compare(picked, color) ? 'selected' : ''"
         :style="{ 'background-color': `${color}` }"
         @click="selectColor(color)"
       >
-        {{ compare(typeof picked === 'string' ? picked : picked.hex8, color) ? '&#10003;' : '' }}
+        {{ compare(picked, color) ? '&#10003;' : '' }}
       </button>
     </div>
     <a-card v-if="props.advanced" class="w-full mt-2" :body-style="{ padding: '0px' }" :bordered="false">
@@ -64,7 +64,7 @@ watch(picked, (n, _o) => {
             Pick Color
           </a-button>
           <div class="flex justify-center py-4">
-            <Chrome v-model="picked" class="!w-full !shadow-none" />
+            <GeneralChromeWrapper v-model="picked" class="!w-full !shadow-none" />
           </div>
         </a-collapse-panel>
       </a-collapse>
diff --git a/packages/nc-gui-v2/composables/useProject.ts b/packages/nc-gui-v2/composables/useProject.ts
index da9f6d1b92..57bcbb0302 100644
--- a/packages/nc-gui-v2/composables/useProject.ts
+++ b/packages/nc-gui-v2/composables/useProject.ts
@@ -10,7 +10,7 @@ const [setup, use] = useInjectionState((_projectId?: MaybeRef<string>) => {
   const { $api } = useNuxtApp()
   const route = useRoute()
   const { includeM2M } = useGlobal()
-  const { setTheme } = useTheme()
+  const { setTheme, theme } = useTheme()
 
   const projectId = computed(() => (_projectId ? unref(_projectId) : (route.params.projectId as string)))
   const project = ref<ProjectType>({})
@@ -99,15 +99,21 @@ const [setup, use] = useInjectionState((_projectId?: MaybeRef<string>) => {
     }
   }
 
-  async function saveTheme(theme: Partial<ThemeConfig>) {
+  async function saveTheme(_theme: Partial<ThemeConfig>) {
+    const fullTheme = {
+      primaryColor: theme.value.primaryColor,
+      accentColor: theme.value.accentColor,
+      ..._theme,
+    }
+
     await updateProject({
-      color: theme.primaryColor,
+      color: fullTheme.primaryColor,
       meta: {
         ...projectMeta.value,
-        theme,
+        theme: fullTheme,
       },
     })
-    setTheme(theme)
+    setTheme(fullTheme)
   }
 
   watch(
diff --git a/packages/nc-gui-v2/pages/[projectType]/[projectId]/index.vue b/packages/nc-gui-v2/pages/[projectType]/[projectId]/index.vue
index d6954d9c05..45120dfbd5 100644
--- a/packages/nc-gui-v2/pages/[projectType]/[projectId]/index.vue
+++ b/packages/nc-gui-v2/pages/[projectType]/[projectId]/index.vue
@@ -1,6 +1,5 @@
 <script setup lang="ts">
 import { message } from 'ant-design-vue'
-import { Chrome } from '@ckpack/vue-color'
 import tinycolor from 'tinycolor2'
 import { useI18n } from 'vue-i18n'
 import {
@@ -20,7 +19,6 @@ import {
   useTabs,
   useTheme,
   useUIPermission,
-  watch,
 } from '#imports'
 import { TabType } from '~/composables'
 
@@ -58,8 +56,6 @@ const sidebar = ref()
 
 const email = computed(() => user.value?.email ?? '---')
 
-const { theme } = useTheme()
-
 const logout = () => {
   signOut()
   navigateTo('/signin')
@@ -84,37 +80,41 @@ await loadProject()
 
 await loadTables()
 
-const themePrimaryColor = ref<any>(theme.value.primaryColor)
-
-const themeAccentColor = ref<any>(theme.value.accentColor)
-
 const { t } = useI18n()
 
-// Chrome provides object so if custom picker used we only edit primary otherwise use complement as accent
-watch(themePrimaryColor, async (nextColor) => {
-  const hexColor = nextColor.hex8 ? nextColor.hex8 : nextColor
-  const tcolor = tinycolor(hexColor)
-  if (tcolor) {
-    const complement = tcolor.complement()
-    await saveTheme({
-      primaryColor: hexColor,
-      accentColor: themeAccentColor.value,
-    })
-    themeAccentColor.value = nextColor.hex8 ? theme.value.accentColor : complement.toHex8String()
+const handleThemeColor = async (mode: 'swatch' | 'primary' | 'accent', color: string) => {
+  switch (mode) {
+    case 'swatch': {
+      const tcolor = tinycolor(color)
+      if (tcolor.isValid()) {
+        const complement = tcolor.complement()
+        await saveTheme({
+          primaryColor: color,
+          accentColor: complement.toHex8String(),
+        })
+      }
+      break
+    }
+    case 'primary': {
+      const tcolor = tinycolor(color)
+      if (tcolor.isValid()) {
+        await saveTheme({
+          primaryColor: color,
+        })
+      }
+      break
+    }
+    case 'accent': {
+      const tcolor = tinycolor(color)
+      if (tcolor.isValid()) {
+        await saveTheme({
+          accentColor: color,
+        })
+      }
+      break
+    }
   }
-})
-
-watch(themeAccentColor, (nextColor) => {
-  const hexColor = nextColor.hex8 ? nextColor.hex8 : nextColor
-
-  // skip if the color is same as saved
-  if (hexColor === theme.value.accentColor) return
-
-  saveTheme({
-    primaryColor: theme.value.primaryColor,
-    accentColor: hexColor,
-  })
-})
+}
 
 if (!route.params.type && isUIAllowed('teamAndAuth')) {
   addTab({ type: TabType.AUTH, title: t('title.teamAndAuth') })
@@ -292,10 +292,10 @@ const copyAuthToken = async () => {
                         <template #expandIcon></template>
 
                         <GeneralColorPicker
-                          v-model="themePrimaryColor"
                           :colors="projectThemeColors"
                           :row-size="9"
                           :advanced="false"
+                          @input="handleThemeColor('swatch', $event)"
                         />
 
                         <!-- Custom Theme -->
@@ -322,7 +322,7 @@ const copyAuthToken = async () => {
                               </div>
                             </template>
                             <template #expandIcon></template>
-                            <Chrome v-model="themePrimaryColor" />
+                            <GeneralChromeWrapper @input="handleThemeColor('primary', $event)" />
                           </a-sub-menu>
 
                           <!-- Accent Color -->
@@ -334,7 +334,7 @@ const copyAuthToken = async () => {
                               </div>
                             </template>
                             <template #expandIcon></template>
-                            <Chrome v-model="themeAccentColor" />
+                            <GeneralChromeWrapper @input="handleThemeColor('accent', $event)" />
                           </a-sub-menu>
                         </a-sub-menu>
                       </a-sub-menu>
diff --git a/packages/nc-gui-v2/pages/index/index/index.vue b/packages/nc-gui-v2/pages/index/index/index.vue
index 3574385e55..f1fab12607 100644
--- a/packages/nc-gui-v2/pages/index/index/index.vue
+++ b/packages/nc-gui-v2/pages/index/index/index.vue
@@ -1,7 +1,6 @@
 <script lang="ts" setup>
 import { Empty, Modal, message } from 'ant-design-vue'
 import type { ProjectType } from 'nocodb-sdk'
-import { Chrome } from '@ckpack/vue-color'
 import tinycolor from 'tinycolor2'
 import {
   computed,
@@ -68,50 +67,24 @@ const deleteProject = (project: ProjectType) => {
 
 await loadProjects()
 
-const themePrimaryColors = $ref(
-  (() => {
-    const colors: Record<string, any> = {}
-    for (const project of projects?.value || []) {
-      if (project?.id) {
-        try {
-          const projectMeta = typeof project.meta === 'string' ? JSON.parse(project.meta) : project.meta
-          colors[project.id] = tinycolor(projectMeta?.theme?.primaryColor).isValid()
-            ? projectMeta?.theme?.primaryColor
-            : themeV2Colors['royal-blue'].DEFAULT
-        } catch (e) {
-          colors[project.id] = themeV2Colors['royal-blue'].DEFAULT
-        }
-      }
-    }
-    return colors
-  })(),
-)
-
-const oldPrimaryColors = ref({ ...themePrimaryColors })
-
-watch(themePrimaryColors, async (nextColors) => {
-  for (const [projectId, nextColor] of Object.entries(nextColors)) {
-    if (oldPrimaryColors.value[projectId] === nextColor) continue
-    const hexColor = nextColor.hex8 ? nextColor.hex8 : nextColor
-    const tcolor = tinycolor(hexColor)
-    if (tcolor) {
-      const complement = tcolor.complement()
-      const project: ProjectType = await $api.project.read(projectId)
-      const meta = project?.meta && typeof project.meta === 'string' ? JSON.parse(project.meta) : project.meta || {}
-      await $api.project.update(projectId, {
-        color: hexColor,
-        meta: JSON.stringify({
-          ...meta,
-          theme: {
-            primaryColor: hexColor,
-            accentColor: complement.toHex8String(),
-          },
-        }),
-      })
-    }
+const handleProjectColor = async (projectId: string, color: string) => {
+  const tcolor = tinycolor(color)
+  if (tcolor.isValid()) {
+    const complement = tcolor.complement()
+    const project: ProjectType = await $api.project.read(projectId)
+    const meta = project?.meta && typeof project.meta === 'string' ? JSON.parse(project.meta) : project.meta || {}
+    await $api.project.update(projectId, {
+      color,
+      meta: JSON.stringify({
+        ...meta,
+        theme: {
+          primaryColor: color,
+          accentColor: complement.toHex8String(),
+        },
+      }),
+    })
   }
-  oldPrimaryColors.value = { ...themePrimaryColors }
-})
+}
 </script>
 
 <template>
@@ -219,7 +192,7 @@ watch(themePrimaryColors, async (nextColors) => {
                         <div
                           class="color-selector"
                           :style="{
-                            'background-color': themePrimaryColors[record.id].hex8 || themePrimaryColors[record.id],
+                            'background-color': record.color || themeV2Colors['royal-blue'].DEFAULT,
                             'width': '8px',
                             'height': '100%',
                           }"
@@ -229,10 +202,10 @@ watch(themePrimaryColors, async (nextColors) => {
                       <template #expandIcon></template>
 
                       <GeneralColorPicker
-                        v-model="themePrimaryColors[record.id]"
                         :colors="projectThemeColors"
                         :row-size="9"
                         :advanced="false"
+                        @input="handleProjectColor(record.id, $event)"
                       />
                       <a-sub-menu key="pick-primary">
                         <template #title>
@@ -242,7 +215,7 @@ watch(themePrimaryColors, async (nextColors) => {
                           </div>
                         </template>
                         <template #expandIcon></template>
-                        <Chrome v-model="themePrimaryColors[record.id]" />
+                        <GeneralChromeWrapper @input="handleProjectColor(record.id, $event)" />
                       </a-sub-menu>
                     </a-sub-menu>
                   </template>