From 2dd68338ed7470d8ae72c13a6f0200ba1a46feda Mon Sep 17 00:00:00 2001
From: AkiraFukushima <h3.poteto@gmail.com>
Date: Wed, 1 Aug 2018 23:09:36 +0900
Subject: [PATCH 1/3] refs #461 Add toot visibility setting in preferences

---
 src/main/preferences.js                       |   3 +-
 .../components/Preferences/General.vue        | 100 ++++++++++++------
 src/renderer/store/Preferences/General.js     |  20 +++-
 3 files changed, 86 insertions(+), 37 deletions(-)

diff --git a/src/main/preferences.js b/src/main/preferences.js
index ac3482b0..3c0b1e1b 100644
--- a/src/main/preferences.js
+++ b/src/main/preferences.js
@@ -9,7 +9,8 @@ const Base = {
     },
     theme: 'white',
     fontSize: 14,
-    displayNameStyle: 0
+    displayNameStyle: 0,
+    tootVisibility: 0
   },
   state: {
     collapse: false
diff --git a/src/renderer/components/Preferences/General.vue b/src/renderer/components/Preferences/General.vue
index 98b6df1d..e7d8dc9b 100644
--- a/src/renderer/components/Preferences/General.vue
+++ b/src/renderer/components/Preferences/General.vue
@@ -34,6 +34,26 @@
       </tbody>
     </table>
   </div>
+  <div class="toot">
+    <h3>Toot</h3>
+    <table>
+      <tbody>
+        <tr>
+          <td class="title">Default Visibility:</td>
+          <td class="status">
+            <el-select v-model="tootVisibility" placeholder="visibility">
+              <el-option
+                v-for="v in visibilities"
+                :key="v.value"
+                :label="v.name"
+                :value="v.value">
+              </el-option>
+            </el-select>
+          </td>
+        </tr>
+      </tbody>
+    </table>
+  </div>
   <div class="sounds">
     <h3>Sounds</h3>
     <table>
@@ -82,6 +102,20 @@ export default {
           name: 'username',
           value: 2
         }
+      ],
+      visibilities: [
+        {
+          name: 'public',
+          value: 0
+        },
+        {
+          name: 'unlisted',
+          value: 1
+        },
+        {
+          name: 'private',
+          value: 2
+        }
       ]
     }
   },
@@ -106,6 +140,14 @@ export default {
         this.$store.dispatch('Preferences/General/updateDisplayNameStyle', value)
       }
     },
+    tootVisibility: {
+      get () {
+        return this.$store.state.Preferences.General.general.tootVisibility
+      },
+      set (value) {
+        this.$store.dispatch('Preferences/General/updateTootVisibility', value)
+      }
+    },
     sound_fav_rb: {
       get () {
         return this.$store.state.Preferences.General.general.sound.fav_rb
@@ -146,52 +188,40 @@ export default {
 
 <style lang="scss" scoped>
 #general {
+  table {
+    width: 100%;
+  }
+
+  td {
+    padding: 16px 0;
+  }
+
+  .title {
+    text-align: right;
+    width: 50%;
+  }
+
+  .status {
+    width: 50%;
+    text-align: center;
+  }
+
   .appearance {
     color: var(--theme-secondary-color);
     width: 100%;
     box-sizing: border-box;
+  }
 
-    table {
-      width: 100%;
-    }
-
-    td {
-      padding: 16px 0;
-    }
-
-    .title {
-      text-align: right;
-      width: 50%;
-    }
-
-    .status {
-      width: 50%;
-      text-align: center;
-    }
+  .toot {
+    color: var(--theme-secondary-color);
+    width: 100%;
+    box-sizing: border-box;
   }
 
   .sounds {
     color: var(--theme-secondary-color);
     width: 100%;
     box-sizing: border-box;
-
-    table {
-      width: 100%;
-    }
-
-    td {
-      padding: 16px 0;
-    }
-
-    .title {
-      text-align: right;
-      width: 50%;
-    }
-
-    .status {
-      width: 50%;
-      text-align: center;
-    }
   }
 }
 </style>
diff --git a/src/renderer/store/Preferences/General.js b/src/renderer/store/Preferences/General.js
index fd3cd3b5..8aa33d33 100644
--- a/src/renderer/store/Preferences/General.js
+++ b/src/renderer/store/Preferences/General.js
@@ -10,7 +10,8 @@ const General = {
       },
       theme: 'white',
       fontSize: 14,
-      displayNameStyle: 0
+      displayNameStyle: 0,
+      tootVisibility: 0
     },
     loading: false
   },
@@ -94,6 +95,23 @@ const General = {
         commit('updateGeneral', conf.general)
       })
     },
+    updateTootVisibility ({ dispatch, commit, state }, value) {
+      const newGeneral = Object.assign({}, state.general, {
+        tootVisibility: value
+      })
+      const config = {
+        general: newGeneral
+      }
+      ipcRenderer.send('save-preferences', config)
+      ipcRenderer.once('error-save-preferences', (event, err) => {
+        ipcRenderer.removeAllListeners('response-save-preferences')
+      })
+      ipcRenderer.once('response-save-preferences', (event, conf) => {
+        ipcRenderer.removeAllListeners('error-save-preferences')
+        dispatch('App/loadPreferences', null, { root: true })
+        commit('updateGeneral', conf.general)
+      })
+    },
     updateSound ({ commit, state }, sound) {
       commit('changeLoading', true)
       const newSound = Object.assign({}, state.general.sound, sound)

From 8638bee417f88f3156da50461c16dbad004d8e9d Mon Sep 17 00:00:00 2001
From: AkiraFukushima <h3.poteto@gmail.com>
Date: Wed, 1 Aug 2018 23:41:05 +0900
Subject: [PATCH 2/3] refs #461 Use constants for visibility

---
 src/constants/visibility.js                   | 18 +++++++++++
 src/main/preferences.js                       |  3 +-
 .../components/Preferences/General.vue        | 16 +++-------
 .../TimelineSpace/Modals/NewToot.vue          | 11 +++----
 src/renderer/store/Preferences/General.js     |  3 +-
 .../store/TimelineSpace/Modals/NewToot.js     | 30 +++++++++++++++----
 6 files changed, 56 insertions(+), 25 deletions(-)
 create mode 100644 src/constants/visibility.js

diff --git a/src/constants/visibility.js b/src/constants/visibility.js
new file mode 100644
index 00000000..a9adca12
--- /dev/null
+++ b/src/constants/visibility.js
@@ -0,0 +1,18 @@
+export default {
+  Public: {
+    name: 'public',
+    value: 0
+  },
+  Unlisted: {
+    name: 'unlisted',
+    value: 1
+  },
+  Private: {
+    name: 'private',
+    value: 2
+  },
+  Direct: {
+    name: 'direct',
+    value: 3
+  }
+}
diff --git a/src/main/preferences.js b/src/main/preferences.js
index 3c0b1e1b..2f8ceddf 100644
--- a/src/main/preferences.js
+++ b/src/main/preferences.js
@@ -1,5 +1,6 @@
 import storage from 'electron-json-storage'
 import objectAssignDeep from 'object-assign-deep'
+import Visibility from '../constants/visibility'
 
 const Base = {
   general: {
@@ -10,7 +11,7 @@ const Base = {
     theme: 'white',
     fontSize: 14,
     displayNameStyle: 0,
-    tootVisibility: 0
+    tootVisibility: Visibility.Public.value
   },
   state: {
     collapse: false
diff --git a/src/renderer/components/Preferences/General.vue b/src/renderer/components/Preferences/General.vue
index e7d8dc9b..de571754 100644
--- a/src/renderer/components/Preferences/General.vue
+++ b/src/renderer/components/Preferences/General.vue
@@ -84,6 +84,7 @@
 
 <script>
 import { mapState } from 'vuex'
+import Visibility from '../../../constants/visibility'
 
 export default {
   name: 'general',
@@ -104,18 +105,9 @@ export default {
         }
       ],
       visibilities: [
-        {
-          name: 'public',
-          value: 0
-        },
-        {
-          name: 'unlisted',
-          value: 1
-        },
-        {
-          name: 'private',
-          value: 2
-        }
+        Visibility.Public,
+        Visibility.Unlisted,
+        Visibility.Private
       ]
     }
   },
diff --git a/src/renderer/components/TimelineSpace/Modals/NewToot.vue b/src/renderer/components/TimelineSpace/Modals/NewToot.vue
index b736b59d..d12973bc 100644
--- a/src/renderer/components/TimelineSpace/Modals/NewToot.vue
+++ b/src/renderer/components/TimelineSpace/Modals/NewToot.vue
@@ -55,6 +55,7 @@
 
 <script>
 import { mapState } from 'vuex'
+import Visibility from '../../../../constants/visibility'
 
 export default {
   name: 'new-toot',
@@ -79,13 +80,13 @@ export default {
       sensitive: state => state.TimelineSpace.Modals.NewToot.sensitive,
       visibilityIcon: (state) => {
         switch (state.TimelineSpace.Modals.NewToot.visibility) {
-          case 'public':
+          case Visibility.Public.value:
             return 'globe'
-          case 'unlisted':
+          case Visibility.Unlisted.value:
             return 'unlock'
-          case 'private':
+          case Visibility.Private.value:
             return 'lock'
-          case 'direct':
+          case Visibility.Direct.value:
             return 'envelope'
           default:
             return 'globe'
@@ -209,7 +210,7 @@ export default {
       this.$store.commit('TimelineSpace/Modals/NewToot/removeMedia', media)
     },
     changeVisibility (level) {
-      this.$store.commit('TimelineSpace/Modals/NewToot/changeVisibility', level)
+      this.$store.dispatch('TimelineSpace/Modals/NewToot/changeVisibility', level)
     },
     changeSensitive () {
       this.$store.commit('TimelineSpace/Modals/NewToot/changeSensitive', !this.sensitive)
diff --git a/src/renderer/store/Preferences/General.js b/src/renderer/store/Preferences/General.js
index 8aa33d33..e630c96a 100644
--- a/src/renderer/store/Preferences/General.js
+++ b/src/renderer/store/Preferences/General.js
@@ -1,4 +1,5 @@
 import { ipcRenderer } from 'electron'
+import Visibility from '../../../constants/visibility'
 
 const General = {
   namespaced: true,
@@ -11,7 +12,7 @@ const General = {
       theme: 'white',
       fontSize: 14,
       displayNameStyle: 0,
-      tootVisibility: 0
+      tootVisibility: Visibility.Public.value
     },
     loading: false
   },
diff --git a/src/renderer/store/TimelineSpace/Modals/NewToot.js b/src/renderer/store/TimelineSpace/Modals/NewToot.js
index 24e7f7c1..0742b3ef 100644
--- a/src/renderer/store/TimelineSpace/Modals/NewToot.js
+++ b/src/renderer/store/TimelineSpace/Modals/NewToot.js
@@ -1,5 +1,6 @@
 import Mastodon from 'megalodon'
 import { ipcRenderer } from 'electron'
+import Visibility from '../../../../constants/visibility'
 
 const NewToot = {
   namespaced: true,
@@ -9,7 +10,7 @@ const NewToot = {
     replyToMessage: null,
     blockSubmit: false,
     attachedMedias: [],
-    visibility: 'public',
+    visibility: Visibility.Public.value,
     sensitive: false,
     spoiler: '',
     attachedMediaId: 0
@@ -36,8 +37,13 @@ const NewToot = {
     removeMedia (state, media) {
       state.attachedMedias = state.attachedMedias.filter(m => m.id !== media.id)
     },
-    changeVisibility (state, value) {
-      state.visibility = value
+    /**
+     * changeVisibility
+     * @param state vuex state object
+     * @param visibility Visibility constants object
+     **/
+    changeVisibility (state, visibility) {
+      state.visibility = visibility.value
     },
     changeSensitive (state, value) {
       state.sensitive = value
@@ -64,14 +70,14 @@ const NewToot = {
           return res.data
         })
     },
-    openReply ({ commit, rootState }, message) {
+    openReply ({ dispatch, commit, rootState }, message) {
       commit('setReplyTo', message)
       const mentionAccounts = [message.account.acct].concat(message.mentions.map(a => a.acct))
         .filter((a, i, self) => self.indexOf(a) === i)
         .filter((a) => a !== rootState.TimelineSpace.account.username)
       commit('changeModal', true)
       commit('updateStatus', `${mentionAccounts.map(m => `@${m}`).join(' ')} `)
-      commit('changeVisibility', message.visibility)
+      dispatch('changeVisibility', message.visibility)
     },
     openModal ({ commit }) {
       commit('changeModal', true)
@@ -84,7 +90,7 @@ const NewToot = {
       commit('clearAttachedMedias')
       commit('changeSensitive', false)
       commit('updateSpoiler', '')
-      commit('changeVisibility', 'public')
+      commit('changeVisibility', Visibility.Public)
     },
     uploadImage ({ state, commit, rootState }, image) {
       commit('changeBlockSubmit', true)
@@ -115,6 +121,18 @@ const NewToot = {
     },
     resetMediaId ({ commit }) {
       commit('updateMediaId', 0)
+    },
+    /**
+     * changeVisibility
+     * @param commit vuex commit object
+     * @param level visibility level string object
+     **/
+    changeVisibility ({ commit }, level) {
+      Object.keys(Visibility).map((key, index) => {
+        if (Visibility[key].name === level) {
+          commit('changeVisibility', Visibility[key])
+        }
+      })
     }
   }
 }

From af43064700140a28d7924c0f8c7046d00b5fbde4 Mon Sep 17 00:00:00 2001
From: AkiraFukushima <h3.poteto@gmail.com>
Date: Thu, 2 Aug 2018 00:02:44 +0900
Subject: [PATCH 3/3] refs #461 Set default visibility from preferences in new
 toot modal

---
 .../components/TimelineSpace/Modals/NewToot.vue  |  5 ++++-
 .../store/TimelineSpace/Modals/NewToot.js        | 16 +++++++++++++++-
 2 files changed, 19 insertions(+), 2 deletions(-)

diff --git a/src/renderer/components/TimelineSpace/Modals/NewToot.vue b/src/renderer/components/TimelineSpace/Modals/NewToot.vue
index d12973bc..dc579a10 100644
--- a/src/renderer/components/TimelineSpace/Modals/NewToot.vue
+++ b/src/renderer/components/TimelineSpace/Modals/NewToot.vue
@@ -147,9 +147,12 @@ export default {
           type: 'error'
         })
       }
+      const visibilityKey = Object.keys(Visibility).find((key) => {
+        return Visibility[key].value === this.visibility
+      })
       let form = {
         status: this.status,
-        visibility: this.visibility,
+        visibility: Visibility[visibilityKey].name,
         sensitive: this.sensitive,
         spoiler_text: this.spoiler
       }
diff --git a/src/renderer/store/TimelineSpace/Modals/NewToot.js b/src/renderer/store/TimelineSpace/Modals/NewToot.js
index 0742b3ef..fe882b1c 100644
--- a/src/renderer/store/TimelineSpace/Modals/NewToot.js
+++ b/src/renderer/store/TimelineSpace/Modals/NewToot.js
@@ -39,12 +39,22 @@ const NewToot = {
     },
     /**
      * changeVisibility
+     * Update visibility using Visibility constants
      * @param state vuex state object
      * @param visibility Visibility constants object
      **/
     changeVisibility (state, visibility) {
       state.visibility = visibility.value
     },
+    /**
+     * changeVisibilityValue
+     * Update visibility using direct value
+     * @param state vuex state object
+     * @param value visibility value
+     **/
+    changeVisibilityValue (state, value) {
+      state.visibility = value
+    },
     changeSensitive (state, value) {
       state.sensitive = value
     },
@@ -79,8 +89,12 @@ const NewToot = {
       commit('updateStatus', `${mentionAccounts.map(m => `@${m}`).join(' ')} `)
       dispatch('changeVisibility', message.visibility)
     },
-    openModal ({ commit }) {
+    openModal ({ dispatch, commit }) {
       commit('changeModal', true)
+      ipcRenderer.send('get-preferences')
+      ipcRenderer.once('response-get-preferences', (event, conf) => {
+        commit('changeVisibilityValue', conf.general.tootVisibility)
+      })
     },
     closeModal ({ commit }) {
       commit('changeModal', false)