From acf3fd5d91b41cfed08d70c63eab7feb79769a10 Mon Sep 17 00:00:00 2001
From: Shpuld Shpuldson <shpuld@gmail.com>
Date: Wed, 2 Aug 2017 22:09:40 +0300
Subject: [PATCH 1/4] Add new user-settings component/route, add options to
 change name, bio, avatar, banner and bg. Add those options to api service and
 backend interactor service.

---
 src/App.scss                                  |   5 +
 .../user_card_content/user_card_content.vue   |   8 +
 src/components/user_panel/user_panel.vue      |   1 +
 src/components/user_settings/user_settings.js | 145 ++++++++++++++++++
 .../user_settings/user_settings.vue           |  90 +++++++++++
 src/main.js                                   |   4 +-
 src/services/api/api.service.js               |  68 ++++++++
 .../backend_interactor_service.js             |   7 +
 8 files changed, 327 insertions(+), 1 deletion(-)
 create mode 100644 src/components/user_settings/user_settings.js
 create mode 100644 src/components/user_settings/user_settings.vue

diff --git a/src/App.scss b/src/App.scss
index 15c908d938..23f7304fe5 100644
--- a/src/App.scss
+++ b/src/App.scss
@@ -42,6 +42,11 @@ button{
     &:hover {
         box-shadow: 0px 0px 4px rgba(255, 255, 255, 0.3);
     }
+
+    &:disabled {
+      cursor: not-allowed;
+      opacity: 0.5;
+    }
 }
 
 .container {
diff --git a/src/components/user_card_content/user_card_content.vue b/src/components/user_card_content/user_card_content.vue
index ff1b108c57..1c044f9ae4 100644
--- a/src/components/user_card_content/user_card_content.vue
+++ b/src/components/user_card_content/user_card_content.vue
@@ -2,6 +2,9 @@
   <div id="heading" class="profile-panel-background" :style="headingStyle">
     <div class="panel-heading text-center">
       <div class='user-info'>
+        <router-link to='/user-settings' style="float: right;">
+          <i class="icon-cog usersettings"></i>
+        </router-link>
         <div class='container'>
           <img :src="user.profile_image_url">
           <span class="glyphicon glyphicon-user"></span>
@@ -128,6 +131,11 @@
   padding: 16px 16px 16px 16px;
   margin-bottom: -4em;
 
+  .usersettings {
+    color: white;
+    opacity: 0.8;
+  }
+
   .container{
     display: flex;
     flex-wrap: wrap;
diff --git a/src/components/user_panel/user_panel.vue b/src/components/user_panel/user_panel.vue
index c5a5ff43d3..69da422c8f 100644
--- a/src/components/user_panel/user_panel.vue
+++ b/src/components/user_panel/user_panel.vue
@@ -6,6 +6,7 @@
 
       <div class="panel-footer base00-background">
         <post-status-form v-if='user'></post-status-form>
+
       </div>
     </div>
     <login-form v-if='!user'></login-form>
diff --git a/src/components/user_settings/user_settings.js b/src/components/user_settings/user_settings.js
new file mode 100644
index 0000000000..c22411dd94
--- /dev/null
+++ b/src/components/user_settings/user_settings.js
@@ -0,0 +1,145 @@
+import StyleSwitcher from '../style_switcher/style_switcher.vue'
+import { filter, trim } from 'lodash'
+
+const UserSettings = {
+  data () {
+    return {
+      newname: this.$store.state.users.currentUser.name,
+      newbio: this.$store.state.users.currentUser.description,
+      previewavatar: null,
+      previewbanner: null,
+      previewbg: null,
+      uploadingavatar: false,
+      uploadingbanner: false,
+      uploadingbg: false
+    }
+  },
+  components: {
+    StyleSwitcher
+  },
+  computed: {
+    user () {
+      return this.$store.state.users.currentUser
+    }
+  },
+  methods: {
+    updateProfile () {
+      const name = this.newname
+      const description = this.newbio
+      this.$store.state.api.backendInteractor.updateProfile({params: {name, description}}).then((user) => {
+        if(!user.error) {
+          this.$store.commit('addNewUsers', [user])
+          this.$store.commit('setCurrentUser', user)
+        }
+      })
+    },
+    uploadAvatar ({target}) {
+      const file = target.files[0]
+      // eslint-disable-next-line no-undef
+      const reader = new FileReader()
+      reader.onload = ({target}) => {
+        const img = target.result
+        this.previewavatar = img
+      }
+      reader.readAsDataURL(file)
+    },
+    uploadBanner ({target}) {
+      const file = target.files[0]
+      // eslint-disable-next-line no-undef
+      const reader = new FileReader()
+      reader.onload = ({target}) => {
+        const img = target.result
+        this.previewbanner = img
+      }
+      reader.readAsDataURL(file)
+    },
+    uploadBg ({target}) {
+      const file = target.files[0]
+      // eslint-disable-next-line no-undef
+      const reader = new FileReader()
+      reader.onload = ({target}) => {
+        const img = target.result
+        this.previewbg = img
+      }
+      reader.readAsDataURL(file)
+    },
+    submitAvatar () {
+      if (!this.previewavatar) { return }
+
+      let img = this.previewavatar
+      // eslint-disable-next-line no-undef
+      let imginfo = new Image()
+      let cropX, cropY, cropW, cropH
+      imginfo.src = img
+      if (imginfo.height > imginfo.width) {
+        cropX = 0
+        cropW = imginfo.width
+        cropY = Math.floor((imginfo.height - imginfo.width) / 2)
+        cropH = imginfo.width
+      } else {
+        cropY = 0
+        cropH = imginfo.height
+        cropX = Math.floor((imginfo.width - imginfo.height) / 2)
+        cropW = imginfo.height
+      }
+      this.uploadingavatar = true
+      this.$store.state.api.backendInteractor.updateAvatar({params: {img, cropX, cropY, cropW, cropH}}).then((user) => {
+        if (!user.error) {
+          this.$store.commit('addNewUsers', [user])
+          this.$store.commit('setCurrentUser', user)
+          this.previewavatar = null
+        }
+        this.uploadingavatar = false
+      })
+    },
+    submitBanner () {
+      if (!this.previewbanner) { return }
+
+      let banner  = this.previewbanner
+      let imginfo = new Image()
+      let offset_top, offset_left, width, height
+      imginfo.src = banner
+      width = imginfo.width
+      height = imginfo.height
+      offset_top = 0
+      offset_left = 0
+      this.uploadingbanner = true
+      this.$store.state.api.backendInteractor.updateBanner({params: {banner, offset_top, offset_left, width, height}}).then((data) => {
+        if (!data.error) {
+          let clone = JSON.parse(JSON.stringify(this.$store.state.users.currentUser))
+          clone.cover_photo = data.url
+          this.$store.commit('addNewUsers', [clone])
+          this.$store.commit('setCurrentUser', clone)
+          this.previewbanner = null
+        }
+        this.uploadingbanner = false
+      })
+    },
+    submitBg () {
+      if (!this.previewbg) { return }
+      let img = this.previewbg
+      let imginfo = new Image()
+      let cropX, cropY, cropW, cropH
+      imginfo.src = img
+      cropX = 0
+      cropY = 0
+      cropW = imginfo.width
+      cropH = imginfo.width
+      this.uploadingbg = true
+      this.$store.state.api.backendInteractor.updateBg({params: {img, cropX, cropY, cropW, cropH}}).then((data) => {
+        if (!data.error) {
+          let clone = JSON.parse(JSON.stringify(this.$store.state.users.currentUser))
+          clone.background_image = data.url
+          this.$store.commit('addNewUsers', [clone])
+          this.$store.commit('setCurrentUser', clone)
+          this.previewbg = null
+        }
+        this.uploadingbg = false
+      })
+    }
+  },
+  watch: {
+  }
+}
+
+export default UserSettings
diff --git a/src/components/user_settings/user_settings.vue b/src/components/user_settings/user_settings.vue
new file mode 100644
index 0000000000..b7521a32b6
--- /dev/null
+++ b/src/components/user_settings/user_settings.vue
@@ -0,0 +1,90 @@
+<template>
+  <div class="settings panel panel-default base00-background">
+    <div class="panel-heading base01-background base04">
+      User Settings
+    </div>
+    <div class="panel-body profile-edit">
+      <div class="setting-item">
+        <h3>Name & Bio</h3>
+        <p>Name</p>
+        <input class='name-changer base03-border' id='username' v-model="newname" :value="user.screen_name"></input>
+        <p>Bio</p>
+        <textarea class="bio base03-border" v-model="newbio"></textarea>
+        <button :disabled='newname.length <= 0' class="btn btn-default base05 base01-background" @click="updateProfile">Submit</button>
+      </div>
+      <div class="setting-item">
+        <h3>Avatar</h3>
+        <p>Your current avatar:</p>
+        <img :src="user.profile_image_url_original" class="old-avatar"></img>
+        <p>Set new avatar:</p>
+        <img class="new-avatar" v-bind:src="previewavatar" v-if="previewavatar">
+        </img>
+        <div>
+          <input type="file" @change="uploadAvatar" ></input>
+        </div>
+        <i class="fa icon-spin4 animate-spin" v-if="uploadingavatar"></i>
+        <button class="btn btn-default base05 base01-background" v-else-if="previewavatar" @click="submitAvatar">Submit</button>
+      </div>
+      <div class="setting-item">
+        <h3>Profile Banner</h3>
+        <p>Your current profile banner:</p>
+        <img :src="user.cover_photo" class="banner"></img>
+        <p>Set new profile banner:</p>
+        <img class="banner" v-bind:src="previewbanner" v-if="previewbanner">
+        </img>
+        <div>
+          <input type="file" @change="uploadBanner" ></input>
+        </div>
+        <i class="fa icon-spin4 animate-spin uploading" v-if="uploadingbanner"></i>
+        <button class="btn btn-default base05 base01-background" v-else-if="previewbanner" @click="submitBanner">Submit</button>
+      </div>
+      <div class="setting-item">
+        <h3>Profile Background</h3>
+        <p>Set new profile background:</p>
+        <img class="bg" v-bind:src="previewbg" v-if="previewbg">
+        </img>
+        <div>
+          <input type="file" @change="uploadBg" ></input>
+        </div>
+        <i class="fa icon-spin4 animate-spin uploading" v-if="uploadingbg"></i>
+        <button class="btn btn-default base05 base01-background" v-else-if="previewbg" @click="submitBg">Submit</button>
+      </div>
+    </div>
+  </div>
+</template>
+
+<script src="./user_settings.js">
+</script>
+
+<style lang="scss">
+.profile-edit {
+  .name-changer {
+    border-width: 1px;
+    border-style: solid;
+    border-radius: 5px;
+    padding: 0.2em 0.2em 0.2em 0.2em;
+  }
+  .name-submit {
+    padding: 0.2em 0.5em 0.2em 0.5em;
+  }
+  .bio {
+    border-width: 1px;
+    border-style: solid;
+    border-radius: 5px;
+    margin: 0;
+  }
+  .banner {
+    max-width: 400px;
+    border-radius: 5px;
+  }
+
+  .bg {
+    max-width: 400px;
+    border-radius: 5px;
+  }
+  .uploading {
+    font-size: 1.5em;
+    margin: 0.25em;
+  }
+}
+</style>
diff --git a/src/main.js b/src/main.js
index 4b7891ed06..8d3a6775ed 100644
--- a/src/main.js
+++ b/src/main.js
@@ -10,6 +10,7 @@ import Mentions from './components/mentions/mentions.vue'
 import UserProfile from './components/user_profile/user_profile.vue'
 import Settings from './components/settings/settings.vue'
 import Registration from './components/registration/registration.vue'
+import UserSettings from './components/user_settings/user_settings.vue'
 
 import statusesModule from './modules/statuses.js'
 import usersModule from './modules/users.js'
@@ -62,7 +63,8 @@ const routes = [
   { name: 'user-profile', path: '/users/:id', component: UserProfile },
   { name: 'mentions', path: '/:username/mentions', component: Mentions },
   { name: 'settings', path: '/settings', component: Settings },
-  { name: 'registration', path: '/registration', component: Registration }
+  { name: 'registration', path: '/registration', component: Registration },
+  { name: 'user-settings', path: '/user-settings', component: UserSettings }
 ]
 
 const router = new VueRouter({
diff --git a/src/services/api/api.service.js b/src/services/api/api.service.js
index e848d076be..b88447d555 100644
--- a/src/services/api/api.service.js
+++ b/src/services/api/api.service.js
@@ -19,6 +19,9 @@ const UNFOLLOWING_URL = '/api/friendships/destroy.json'
 const QVITTER_USER_PREF_URL = '/api/qvitter/set_profile_pref.json'
 const REGISTRATION_URL = '/api/account/register.json'
 const AVATAR_UPDATE_URL = '/api/qvitter/update_avatar.json'
+const BG_UPDATE_URL = '/api/qvitter/update_background_image.json'
+const BANNER_UPDATE_URL = '/api/account/update_profile_banner.json'
+const PROFILE_UPDATE_URL = '/api/account/update_profile.json'
 const EXTERNAL_PROFILE_URL = '/api/externalprofile/show.json'
 const QVITTER_USER_TIMELINE_URL = '/api/qvitter/statuses/user_timeline.json'
 // const USER_URL = '/api/users/show.json'
@@ -56,6 +59,68 @@ const updateAvatar = ({credentials, params}) => {
   }).then((data) => data.json())
 }
 
+const updateBg = ({credentials, params}) => {
+  let url = BG_UPDATE_URL
+
+  const form = new FormData()
+
+  each(params, (value, key) => {
+    if (value) {
+      form.append(key, value)
+    }
+  })
+  return fetch(url, {
+    headers: authHeaders(credentials),
+    method: 'POST',
+    body: form
+  }).then((data) => data.json())
+}
+
+// Params
+// height
+// width
+// offset_left
+// offset_top
+// banner (base 64 encodend data url)
+const updateBanner = ({credentials, params}) => {
+  let url = BANNER_UPDATE_URL
+
+  const form = new FormData()
+
+  each(params, (value, key) => {
+    if (value) {
+      form.append(key, value)
+    }
+  })
+  return fetch(url, {
+    headers: authHeaders(credentials),
+    method: 'POST',
+    body: form
+  }).then((data) => data.json())
+}
+
+// Params
+// name
+// url
+// location
+// description
+const updateProfile = ({credentials, params}) => {
+  let url = PROFILE_UPDATE_URL
+
+  const form = new FormData()
+
+  each(params, (value, key) => {
+    if (value) {
+      form.append(key, value)
+    }
+  })
+  return fetch(url, {
+    headers: authHeaders(credentials),
+    method: 'POST',
+    body: form
+  }).then((data) => data.json())
+}
+
 // Params needed:
 // nickname
 // email
@@ -273,6 +338,9 @@ const apiService = {
   fetchMutes,
   register,
   updateAvatar,
+  updateBg,
+  updateProfile,
+  updateBanner,
   externalProfile
 }
 
diff --git a/src/services/backend_interactor_service/backend_interactor_service.js b/src/services/backend_interactor_service/backend_interactor_service.js
index 5dbbf4b37f..5311f26805 100644
--- a/src/services/backend_interactor_service/backend_interactor_service.js
+++ b/src/services/backend_interactor_service/backend_interactor_service.js
@@ -38,6 +38,10 @@ const backendInteractorService = (credentials) => {
 
   const register = (params) => apiService.register(params)
   const updateAvatar = ({params}) => apiService.updateAvatar({credentials, params})
+  const updateBg = ({params}) => apiService.updateBg({credentials, params})
+  const updateBanner = ({params}) => apiService.updateBanner({credentials, params})
+  const updateProfile = ({params}) => apiService.updateProfile({credentials, params})
+
   const externalProfile = (profileUrl) => apiService.externalProfile(profileUrl)
 
   const backendInteractorServiceInstance = {
@@ -53,6 +57,9 @@ const backendInteractorService = (credentials) => {
     fetchMutes,
     register,
     updateAvatar,
+    updateBg,
+    updateBanner,
+    updateProfile,
     externalProfile
   }
 

From 6d7d173e4da4b7d8ffee33e21d8c072efe2f7be9 Mon Sep 17 00:00:00 2001
From: Shpuld Shpuldson <shpuld@gmail.com>
Date: Wed, 16 Aug 2017 00:46:55 +0300
Subject: [PATCH 2/4] Make linter happy.

---
 src/components/user_settings/user_settings.js  | 9 ++++++---
 src/components/user_settings/user_settings.vue | 4 ----
 2 files changed, 6 insertions(+), 7 deletions(-)

diff --git a/src/components/user_settings/user_settings.js b/src/components/user_settings/user_settings.js
index c22411dd94..2f38471117 100644
--- a/src/components/user_settings/user_settings.js
+++ b/src/components/user_settings/user_settings.js
@@ -1,5 +1,4 @@
 import StyleSwitcher from '../style_switcher/style_switcher.vue'
-import { filter, trim } from 'lodash'
 
 const UserSettings = {
   data () {
@@ -27,7 +26,7 @@ const UserSettings = {
       const name = this.newname
       const description = this.newbio
       this.$store.state.api.backendInteractor.updateProfile({params: {name, description}}).then((user) => {
-        if(!user.error) {
+        if (!user.error) {
           this.$store.commit('addNewUsers', [user])
           this.$store.commit('setCurrentUser', user)
         }
@@ -95,8 +94,10 @@ const UserSettings = {
     submitBanner () {
       if (!this.previewbanner) { return }
 
-      let banner  = this.previewbanner
+      let banner = this.previewbanner
+      // eslint-disable-next-line no-undef
       let imginfo = new Image()
+      /* eslint-disable camelcase */
       let offset_top, offset_left, width, height
       imginfo.src = banner
       width = imginfo.width
@@ -114,10 +115,12 @@ const UserSettings = {
         }
         this.uploadingbanner = false
       })
+      /* eslint-enable camelcase */
     },
     submitBg () {
       if (!this.previewbg) { return }
       let img = this.previewbg
+      // eslint-disable-next-line no-undef
       let imginfo = new Image()
       let cropX, cropY, cropW, cropH
       imginfo.src = img
diff --git a/src/components/user_settings/user_settings.vue b/src/components/user_settings/user_settings.vue
index b7521a32b6..49d2c8c91b 100644
--- a/src/components/user_settings/user_settings.vue
+++ b/src/components/user_settings/user_settings.vue
@@ -78,10 +78,6 @@
     border-radius: 5px;
   }
 
-  .bg {
-    max-width: 400px;
-    border-radius: 5px;
-  }
   .uploading {
     font-size: 1.5em;
     margin: 0.25em;

From 1526b4560c9dd6f2fff5fcb24890a398c8701343 Mon Sep 17 00:00:00 2001
From: Shpuld Shpuldson <shpuld@gmail.com>
Date: Fri, 18 Aug 2017 11:51:17 +0300
Subject: [PATCH 3/4] Clean up to have much less redundant code.

---
 src/components/user_settings/user_settings.js | 68 ++++++-------------
 .../user_settings/user_settings.vue           | 24 +++----
 2 files changed, 34 insertions(+), 58 deletions(-)

diff --git a/src/components/user_settings/user_settings.js b/src/components/user_settings/user_settings.js
index 2f38471117..fd20a6ad82 100644
--- a/src/components/user_settings/user_settings.js
+++ b/src/components/user_settings/user_settings.js
@@ -5,12 +5,8 @@ const UserSettings = {
     return {
       newname: this.$store.state.users.currentUser.name,
       newbio: this.$store.state.users.currentUser.description,
-      previewavatar: null,
-      previewbanner: null,
-      previewbg: null,
-      uploadingavatar: false,
-      uploadingbanner: false,
-      uploadingbg: false
+      uploading: [ false, false, false ],
+      previews: [ null, null, null ]
     }
   },
   components: {
@@ -32,40 +28,22 @@ const UserSettings = {
         }
       })
     },
-    uploadAvatar ({target}) {
-      const file = target.files[0]
+    uploadFile (slot, e) {
+      const file = e.target.files[0]
+      if (!file) { return }
       // eslint-disable-next-line no-undef
       const reader = new FileReader()
       reader.onload = ({target}) => {
         const img = target.result
-        this.previewavatar = img
-      }
-      reader.readAsDataURL(file)
-    },
-    uploadBanner ({target}) {
-      const file = target.files[0]
-      // eslint-disable-next-line no-undef
-      const reader = new FileReader()
-      reader.onload = ({target}) => {
-        const img = target.result
-        this.previewbanner = img
-      }
-      reader.readAsDataURL(file)
-    },
-    uploadBg ({target}) {
-      const file = target.files[0]
-      // eslint-disable-next-line no-undef
-      const reader = new FileReader()
-      reader.onload = ({target}) => {
-        const img = target.result
-        this.previewbg = img
+        this.previews[slot] = img
+        this.$forceUpdate() // just changing the array with the index doesn't update the view
       }
       reader.readAsDataURL(file)
     },
     submitAvatar () {
-      if (!this.previewavatar) { return }
+      if (!this.previews[0]) { return }
 
-      let img = this.previewavatar
+      let img = this.previews[0]
       // eslint-disable-next-line no-undef
       let imginfo = new Image()
       let cropX, cropY, cropW, cropH
@@ -81,20 +59,20 @@ const UserSettings = {
         cropX = Math.floor((imginfo.width - imginfo.height) / 2)
         cropW = imginfo.height
       }
-      this.uploadingavatar = true
+      this.uploading[0] = true
       this.$store.state.api.backendInteractor.updateAvatar({params: {img, cropX, cropY, cropW, cropH}}).then((user) => {
         if (!user.error) {
           this.$store.commit('addNewUsers', [user])
           this.$store.commit('setCurrentUser', user)
-          this.previewavatar = null
+          this.previews[0] = null
         }
-        this.uploadingavatar = false
+        this.uploading[0] = false
       })
     },
     submitBanner () {
-      if (!this.previewbanner) { return }
+      if (!this.previews[1]) { return }
 
-      let banner = this.previewbanner
+      let banner = this.previews[1]
       // eslint-disable-next-line no-undef
       let imginfo = new Image()
       /* eslint-disable camelcase */
@@ -104,22 +82,22 @@ const UserSettings = {
       height = imginfo.height
       offset_top = 0
       offset_left = 0
-      this.uploadingbanner = true
+      this.uploading[1] = true
       this.$store.state.api.backendInteractor.updateBanner({params: {banner, offset_top, offset_left, width, height}}).then((data) => {
         if (!data.error) {
           let clone = JSON.parse(JSON.stringify(this.$store.state.users.currentUser))
           clone.cover_photo = data.url
           this.$store.commit('addNewUsers', [clone])
           this.$store.commit('setCurrentUser', clone)
-          this.previewbanner = null
+          this.previews[1] = null
         }
-        this.uploadingbanner = false
+        this.uploading[1] = false
       })
       /* eslint-enable camelcase */
     },
     submitBg () {
-      if (!this.previewbg) { return }
-      let img = this.previewbg
+      if (!this.previews[2]) { return }
+      let img = this.previews[2]
       // eslint-disable-next-line no-undef
       let imginfo = new Image()
       let cropX, cropY, cropW, cropH
@@ -128,20 +106,18 @@ const UserSettings = {
       cropY = 0
       cropW = imginfo.width
       cropH = imginfo.width
-      this.uploadingbg = true
+      this.uploading[2] = true
       this.$store.state.api.backendInteractor.updateBg({params: {img, cropX, cropY, cropW, cropH}}).then((data) => {
         if (!data.error) {
           let clone = JSON.parse(JSON.stringify(this.$store.state.users.currentUser))
           clone.background_image = data.url
           this.$store.commit('addNewUsers', [clone])
           this.$store.commit('setCurrentUser', clone)
-          this.previewbg = null
+          this.previews[2] = null
         }
-        this.uploadingbg = false
+        this.uploading[2] = false
       })
     }
-  },
-  watch: {
   }
 }
 
diff --git a/src/components/user_settings/user_settings.vue b/src/components/user_settings/user_settings.vue
index 49d2c8c91b..11da79e4cf 100644
--- a/src/components/user_settings/user_settings.vue
+++ b/src/components/user_settings/user_settings.vue
@@ -17,37 +17,37 @@
         <p>Your current avatar:</p>
         <img :src="user.profile_image_url_original" class="old-avatar"></img>
         <p>Set new avatar:</p>
-        <img class="new-avatar" v-bind:src="previewavatar" v-if="previewavatar">
+        <img class="new-avatar" v-bind:src="previews[0]" v-if="previews[0]">
         </img>
         <div>
-          <input type="file" @change="uploadAvatar" ></input>
+          <input type="file" @change="uploadFile(0, $event)" ></input>
         </div>
-        <i class="fa icon-spin4 animate-spin" v-if="uploadingavatar"></i>
-        <button class="btn btn-default base05 base01-background" v-else-if="previewavatar" @click="submitAvatar">Submit</button>
+        <i class="fa icon-spin4 animate-spin" v-if="uploading[0]"></i>
+        <button class="btn btn-default base05 base01-background" v-else-if="previews[0]" @click="submitAvatar">Submit</button>
       </div>
       <div class="setting-item">
         <h3>Profile Banner</h3>
         <p>Your current profile banner:</p>
         <img :src="user.cover_photo" class="banner"></img>
         <p>Set new profile banner:</p>
-        <img class="banner" v-bind:src="previewbanner" v-if="previewbanner">
+        <img class="banner" v-bind:src="previews[1]" v-if="previews[1]">
         </img>
         <div>
-          <input type="file" @change="uploadBanner" ></input>
+          <input type="file" @change="uploadFile(1, $event)" ></input>
         </div>
-        <i class="fa icon-spin4 animate-spin uploading" v-if="uploadingbanner"></i>
-        <button class="btn btn-default base05 base01-background" v-else-if="previewbanner" @click="submitBanner">Submit</button>
+        <i class="fa icon-spin4 animate-spin uploading" v-if="uploading[1]"></i>
+        <button class="btn btn-default base05 base01-background" v-else-if="previews[1]" @click="submitBanner">Submit</button>
       </div>
       <div class="setting-item">
         <h3>Profile Background</h3>
         <p>Set new profile background:</p>
-        <img class="bg" v-bind:src="previewbg" v-if="previewbg">
+        <img class="bg" v-bind:src="previews[2]" v-if="previews[2]">
         </img>
         <div>
-          <input type="file" @change="uploadBg" ></input>
+          <input type="file" @change="uploadFile(2, $event)" ></input>
         </div>
-        <i class="fa icon-spin4 animate-spin uploading" v-if="uploadingbg"></i>
-        <button class="btn btn-default base05 base01-background" v-else-if="previewbg" @click="submitBg">Submit</button>
+        <i class="fa icon-spin4 animate-spin uploading" v-if="uploading[2]"></i>
+        <button class="btn btn-default base05 base01-background" v-else-if="previews[2]" @click="submitBg">Submit</button>
       </div>
     </div>
   </div>

From 9503b8fa41b9f6e3c500af411b29ee1c27844253 Mon Sep 17 00:00:00 2001
From: Shpuld Shpuldson <shpuld@gmail.com>
Date: Fri, 18 Aug 2017 12:00:59 +0300
Subject: [PATCH 4/4] Remove the profile pic change from the old settings menu
 since it's in the new profile edit menu.

---
 src/components/settings/settings.js  | 41 +---------------------------
 src/components/settings/settings.vue | 12 --------
 2 files changed, 1 insertion(+), 52 deletions(-)

diff --git a/src/components/settings/settings.js b/src/components/settings/settings.js
index 8d6a671318..b3bb82909f 100644
--- a/src/components/settings/settings.js
+++ b/src/components/settings/settings.js
@@ -9,8 +9,7 @@ const settings = {
       hideNsfwLocal: this.$store.state.config.hideNsfw,
       muteWordsString: this.$store.state.config.muteWords.join('\n'),
       autoLoadLocal: this.$store.state.config.autoLoad,
-      hoverPreviewLocal: this.$store.state.config.hoverPreview,
-      previewfile: null
+      hoverPreviewLocal: this.$store.state.config.hoverPreview
     }
   },
   components: {
@@ -21,44 +20,6 @@ const settings = {
       return this.$store.state.users.currentUser
     }
   },
-  methods: {
-    uploadAvatar ({target}) {
-      const file = target.files[0]
-      // eslint-disable-next-line no-undef
-      const reader = new FileReader()
-      reader.onload = ({target}) => {
-        const img = target.result
-        this.previewfile = img
-      }
-      reader.readAsDataURL(file)
-    },
-    submitAvatar () {
-      if (!this.previewfile) { return }
-
-      const img = this.previewfile
-      // eslint-disable-next-line no-undef
-      let imginfo = new Image()
-      let cropX, cropY, cropW, cropH
-      imginfo.src = this.previewfile
-      if (imginfo.height > imginfo.width) {
-        cropX = 0
-        cropW = imginfo.width
-        cropY = Math.floor((imginfo.height - imginfo.width) / 2)
-        cropH = imginfo.width
-      } else {
-        cropY = 0
-        cropH = imginfo.height
-        cropX = Math.floor((imginfo.width - imginfo.height) / 2)
-        cropW = imginfo.height
-      }
-      this.$store.state.api.backendInteractor.updateAvatar({params: {img, cropX, cropY, cropW, cropH}}).then((user) => {
-        if (!user.error) {
-          this.$store.commit('addNewUsers', [user])
-          this.$store.commit('setCurrentUser', user)
-        }
-      })
-    }
-  },
   watch: {
     hideAttachmentsLocal (value) {
       this.$store.dispatch('setOption', { name: 'hideAttachments', value })
diff --git a/src/components/settings/settings.vue b/src/components/settings/settings.vue
index 1abb178909..6ff96fece5 100644
--- a/src/components/settings/settings.vue
+++ b/src/components/settings/settings.vue
@@ -8,18 +8,6 @@
         <h2>Theme</h2>
         <style-switcher></style-switcher>
       </div>
-      <div class="setting-item" v-if="user">
-        <h2>Avatar</h2>
-        <p>Your current avatar:</p>
-        <img :src="user.profile_image_url_original" class="old-avatar"></img>
-        <p>Set new avatar:</p>
-        <img class="new-avatar" v-bind:src="previewfile" v-if="previewfile">
-        </img>
-        <div>
-          <input name="avatar-upload" id="avatar-upload" type="file" @change="uploadAvatar" ></input>
-        </div>
-        <button class="btn btn-default base05 base01-background" v-if="previewfile" @click="submitAvatar">Submit</button>
-      </div>
       <div class="setting-item">
         <h2>Filtering</h2>
         <p>All notices containing these words will be muted, one per line</p>