From f18ca59c0629391c787c491006d34fe0eed1ed1a Mon Sep 17 00:00:00 2001
From: AkiraFukushima <h3.poteto@gmail.com>
Date: Wed, 4 May 2022 22:08:55 +0900
Subject: [PATCH] refs #3301 Rewrite Preferences/Appearance with composition
 API

---
 .../components/Preferences/Appearance.vue     | 125 +++++++++---------
 .../Preferences/Appearance/Toot.vue           |   2 +-
 src/renderer/store/Preferences/Appearance.ts  |  10 +-
 3 files changed, 67 insertions(+), 70 deletions(-)

diff --git a/src/renderer/components/Preferences/Appearance.vue b/src/renderer/components/Preferences/Appearance.vue
index 207c8e88..335c1b96 100644
--- a/src/renderer/components/Preferences/Appearance.vue
+++ b/src/renderer/components/Preferences/Appearance.vue
@@ -48,82 +48,79 @@
   </div>
 </template>
 
-<script>
-import { mapState } from 'vuex'
-import Toot from './Appearance/Toot'
-import ColorPallet from './Appearance/ColorPallet'
+<script lang="ts">
+import { defineComponent, computed, onMounted } from 'vue'
+import { useStore } from '@/store'
+import Toot from './Appearance/Toot.vue'
+import ColorPallet from './Appearance/ColorPallet.vue'
 import DisplayStyle from '~/src/constants/displayStyle'
 import Theme from '~/src/constants/theme'
 import TimeFormat from '~/src/constants/timeFormat'
+import { ACTION_TYPES } from '@/store/Preferences/Appearance'
 
-export default {
+export default defineComponent({
   name: 'appearance',
   components: {
     Toot,
     ColorPallet
   },
-  data() {
+  setup() {
+    const space = 'Preferences/Appearance'
+    const store = useStore()
+
+    const nameStyles = [DisplayStyle.DisplayNameAndUsername, DisplayStyle.DisplayName, DisplayStyle.Username]
+    const themes = [Theme.System, Theme.Light, Theme.Dark, Theme.SolarizedLight, Theme.SolarizedDark, Theme.KimbieDark, Theme.Custom]
+    const timeFormats = [TimeFormat.Absolute, TimeFormat.Relative]
+
+    const fontSize = computed(() => store.state.Preferences.Appearance.appearance.fontSize)
+    const fontList = computed(() => store.state.Preferences.Appearance.fonts)
+    const tootPadding = computed(() => store.state.Preferences.Appearance.appearance.tootPadding)
+    const theme = computed({
+      get: () => store.state.Preferences.Appearance.appearance.theme,
+      set: value => store.dispatch(`${space}/${ACTION_TYPES.UPDATE_THEME}`, value)
+    })
+    const displayNameStyle = computed({
+      get: () => store.state.Preferences.Appearance.appearance.displayNameStyle,
+      set: value => store.dispatch(`${space}/${ACTION_TYPES.UPDATE_DISPLAY_NAME_STYLE}`, value)
+    })
+    const timeFormat = computed({
+      get: () => store.state.Preferences.Appearance.appearance.timeFormat,
+      set: value => store.dispatch(`${space}/${ACTION_TYPES.UPDATE_TIME_FORMAT}`, value)
+    })
+    const customizeThemeColor = computed(() => theme.value === Theme.Custom.key)
+    const font = computed({
+      get: () => store.state.Preferences.Appearance.appearance.font,
+      set: value => store.dispatch(`${space}/${ACTION_TYPES.UPDATE_FONT}`, value)
+    })
+
+    onMounted(() => {
+      store.dispatch(`${space}/${ACTION_TYPES.LOAD_APPEARANCE}`)
+      store.dispatch(`${space}/${ACTION_TYPES.LOAD_FONTS}`)
+    })
+
+    const updateFontSize = async (value: number) => {
+      await store.dispatch(`${space}/${ACTION_TYPES.UPDATE_FONT_SIZE}`, value)
+    }
+    const updateTootPadding = async (value: number) => {
+      store.dispatch(`${space}/${ACTION_TYPES.UPDATE_TOOT_PADDING}`, value)
+    }
     return {
-      nameStyles: [DisplayStyle.DisplayNameAndUsername, DisplayStyle.DisplayName, DisplayStyle.Username],
-      themes: [Theme.System, Theme.Light, Theme.Dark, Theme.SolarizedLight, Theme.SolarizedDark, Theme.KimbieDark, Theme.Custom],
-      timeFormats: [TimeFormat.Absolute, TimeFormat.Relative]
-    }
-  },
-  computed: {
-    ...mapState('Preferences/Appearance', {
-      fontSize: state => state.appearance.fontSize,
-      fontList: state => state.fonts,
-      tootPadding: state => state.appearance.tootPadding
-    }),
-    theme: {
-      get() {
-        return this.$store.state.Preferences.Appearance.appearance.theme
-      },
-      set(value) {
-        this.$store.dispatch('Preferences/Appearance/updateTheme', value)
-      }
-    },
-    displayNameStyle: {
-      get() {
-        return this.$store.state.Preferences.Appearance.appearance.displayNameStyle
-      },
-      set(value) {
-        this.$store.dispatch('Preferences/Appearance/updateDisplayNameStyle', value)
-      }
-    },
-    timeFormat: {
-      get() {
-        return this.$store.state.Preferences.Appearance.appearance.timeFormat
-      },
-      set(value) {
-        this.$store.dispatch('Preferences/Appearance/updateTimeFormat', value)
-      }
-    },
-    customizeThemeColor() {
-      return this.theme === Theme.Custom.key
-    },
-    font: {
-      get() {
-        return this.$store.state.Preferences.Appearance.appearance.font
-      },
-      set(value) {
-        this.$store.dispatch('Preferences/Appearance/updateFont', value)
-      }
-    }
-  },
-  created() {
-    this.$store.dispatch('Preferences/Appearance/loadAppearance')
-    this.$store.dispatch('Preferences/Appearance/loadFonts')
-  },
-  methods: {
-    async updateFontSize(value) {
-      await this.$store.dispatch('Preferences/Appearance/updateFontSize', value)
-    },
-    async updateTootPadding(value) {
-      await this.$store.dispatch('Preferences/Appearance/updateTootPadding', value)
+      nameStyles,
+      themes,
+      timeFormats,
+      fontSize,
+      fontList,
+      tootPadding,
+      theme,
+      displayNameStyle,
+      timeFormat,
+      customizeThemeColor,
+      font,
+      updateFontSize,
+      updateTootPadding
     }
   }
-}
+})
 </script>
 
 <style lang="scss" scoped>
diff --git a/src/renderer/components/Preferences/Appearance/Toot.vue b/src/renderer/components/Preferences/Appearance/Toot.vue
index 7e92bb0a..2ac71f96 100644
--- a/src/renderer/components/Preferences/Appearance/Toot.vue
+++ b/src/renderer/components/Preferences/Appearance/Toot.vue
@@ -1,5 +1,5 @@
 <template>
-  <div class="status" tabIndex="0" ref="status">
+  <div class="status" tabIndex="0" ref="statusRef">
     <div class="toot">
       <div class="icon">
         <img :src="sampleIcon" />
diff --git a/src/renderer/store/Preferences/Appearance.ts b/src/renderer/store/Preferences/Appearance.ts
index 6b21aa0b..e909787b 100644
--- a/src/renderer/store/Preferences/Appearance.ts
+++ b/src/renderer/store/Preferences/Appearance.ts
@@ -79,7 +79,7 @@ const actions: ActionTree<AppearanceState, RootState> = {
     dispatch('App/loadPreferences', null, { root: true })
   },
   [ACTION_TYPES.UPDATE_FONT_SIZE]: async ({ dispatch, commit, state }, fontSize: number) => {
-    const newAppearance: Appearance = Object.assign({}, state.appearance, {
+    const newAppearance: Appearance = Object.assign({}, toRaw(state.appearance), {
       fontSize: fontSize
     })
     const config = {
@@ -90,7 +90,7 @@ const actions: ActionTree<AppearanceState, RootState> = {
     dispatch('App/loadPreferences', null, { root: true })
   },
   [ACTION_TYPES.UPDATE_DISPLAY_NAME_STYLE]: async ({ dispatch, commit, state }, value: number) => {
-    const newAppearance: Appearance = Object.assign({}, state.appearance, {
+    const newAppearance: Appearance = Object.assign({}, toRaw(state.appearance), {
       displayNameStyle: value
     })
     const config = {
@@ -101,7 +101,7 @@ const actions: ActionTree<AppearanceState, RootState> = {
     commit(MUTATION_TYPES.UPDATE_APPEARANCE, conf.appearance)
   },
   [ACTION_TYPES.UPDATE_TIME_FORMAT]: async ({ dispatch, commit, state }, value: number) => {
-    const newAppearance: Appearance = Object.assign({}, state.appearance, {
+    const newAppearance: Appearance = Object.assign({}, toRaw(state.appearance), {
       timeFormat: value
     })
     const config = {
@@ -124,7 +124,7 @@ const actions: ActionTree<AppearanceState, RootState> = {
     dispatch('App/loadPreferences', null, { root: true })
   },
   [ACTION_TYPES.UPDATE_FONT]: async ({ dispatch, state, commit }, value: string) => {
-    const newAppearance: Appearance = Object.assign({}, state.appearance, {
+    const newAppearance: Appearance = Object.assign({}, toRaw(state.appearance), {
       font: value
     })
     const config = {
@@ -135,7 +135,7 @@ const actions: ActionTree<AppearanceState, RootState> = {
     dispatch('App/loadPreferences', null, { root: true })
   },
   [ACTION_TYPES.UPDATE_TOOT_PADDING]: async ({ dispatch, state, commit }, value: number) => {
-    const newAppearance: Appearance = Object.assign({}, state.appearance, {
+    const newAppearance: Appearance = Object.assign({}, toRaw(state.appearance), {
       tootPadding: value
     })
     const config = {