fix: expand status changed can not be stored & show more button display

src/_locales/en/messages.json

@@ -119,6 +119,9 @@
   "opt_desc_openTabListNoTab": {
     "message": "Open the tab list when store all tabs"
   },
+  "opt_desc_listsPerPage": {
+    "message": "Number of lists per page"
+  },
   "opt_label_popup": {
     "message": "Popup simple list"
   },

src/_locales/zh_CN/messages.json

@@ -119,6 +119,9 @@
   "opt_desc_openTabListNoTab": {
     "message": "当储存所有标签时打开标签列表"
   },
+  "opt_desc_listsPerPage": {
+    "message": "每页的列表数量"
+  },
   "opt_label_popup": {
     "message": "弹出简单列表"
   },

src/background.js

@@ -5,6 +5,7 @@ import storage from './common/storage'
 import options from './common/options'
 import migrate from './common/migrate'
 import boss from './common/service/boss'
+import listManger from './common/listManager'
 import browser from 'webextension-polyfill'
 
 
@@ -212,6 +213,7 @@ const commandHandler = async command => {
 }
 
 const init = async () => {
+  await listManger.init()
   const opts = window.opts = await storage.getOptions() || {}
   _.defaults(opts, options.getDefaultOptions())
   await storage.setOptions(opts)

src/common/listManager.js

@@ -19,7 +19,6 @@ const compressOps = ops => {
   const removed = []
   const updated = {}
   const finalOps = []
-  console.log(ops)
   for (let i = ops.length - 1; i > -1; i -= 1) {
     const op = ops[i]
     // ignore all actions for the list if that list will be removed finally

src/common/service/boss.js

@@ -5,7 +5,7 @@ import {
 import {isBackground} from '../utils'
 import browser from 'webextension-polyfill'
 
-   const apiUrl = 'http://127.0.0.1:3000'
+const apiUrl = 'http://127.0.0.1:3000' // TODO: use online address
 
 const hasToken = async () => TOKEN_KEY in await browser.storage.local.get(TOKEN_KEY)
 
@@ -107,15 +107,15 @@ const uploadOperations = async () => {
 
 const applyRemoteLists = async () => {
   const lists = await getLists()
-  const {listsUpdatedAt} = browser.storage.local.set({lists})
-  return Date.parse(listsUpdatedAt)
+  await browser.storage.local.set({lists})
+  return getInfo()
 }
 
 const uploadOpts = async () => {
   const {opts} = await browser.storage.local.get('opts')
-  const optsUpdatedAt = await setOpts(opts)
-  await browser.storage.local.set({optsUpdatedAt})
-  return Date.parse(optsUpdatedAt)
+  const result = await setOpts(opts)
+  result.optsUpdatedAt = Date.parse(result.optsUpdatedAt)
+  return result
 }
 
 const applyRemoteOpts = async () => {
@@ -141,12 +141,12 @@ const refresh = async () => {
   }
   // apply remote lists if remote lists update time later than local
   if (remoteInfo.listsUpdatedAt > localInfo.listsUpdatedAt) {
-    const listsUpdatedAt = await applyRemoteLists()
+    const {listsUpdatedAt} = await applyRemoteLists()
     await browser.storage.local.set({listsUpdatedAt})
   }
 
   if (localInfo.optsUpdatedAt > remoteInfo.optsUpdatedAt) {
-    const optsUpdatedAt = await uploadOpts()
+    const {optsUpdatedAt} = await uploadOpts()
     await browser.storage.local.set({optsUpdatedAt})
   } else if (localInfo.optsUpdatedAt < remoteInfo.optsUpdatedAt) {
     await applyRemoteOpts()

src/common/utils.js

@@ -40,6 +40,6 @@ export const genObjectId = () => {
   return timestamp + 'xxxxxxxxxxxxxxxx'.replace(/[x]/g, () => (Math.random() * 16 | 0).toString(16)).toLowerCase()
 }
 export const isBackground = async () => {
-  if (window._isBackground != null) window._isBackground = window === await browser.runtime.getBackgroundPage()
+  if (window._isBackground == null) window._isBackground = window === await browser.runtime.getBackgroundPage()
   return window._isBackground
 }

src/component/main/Toolbar.vue

@@ -9,8 +9,7 @@
   <v-tooltip left>
     <v-btn slot="activator" icon dark :loading="syncing" :disabled="!online" @click="syncBtnClicked">
       <transition name="fade" mode="out-in">
-        <v-icon v-if="uploadSuccess" key="success">cloud_done</v-icon>
-        <v-icon v-else="!uploadSuccess" key="normal">{{ syncIcon }}</v-icon>
+        <v-icon :key="syncIcon">{{ syncIcon }}</v-icon>
       </transition>
     </v-btn>
     <span>{{ tooltip }}<dynamic-time v-if="!tooltip" v-model="lastUpdated"></dynamic-time></span>
@@ -24,7 +23,6 @@
 </v-toolbar>
 </template>
 <script>
-import _ from 'lodash'
 import __ from '@/common/i18n'
 import searchForm from './SearchForm'
 import dynamicTime from '@/component/DynamicTime'
@@ -48,7 +46,7 @@ export default {
   computed: {
     ...mapState(['opts', 'hasToken', 'conflict', 'nightmode']),
     tooltip() {
-      return !this.online ? __('ui_offline')
+      return !this.online ? __('ui_offline') // eslint-disable-line
         : !this.hasToken ? __('ui_not_login') // eslint-disable-line
         : this.syncing ? __('ui_syncing')
         // : this.conflict ? __('ui_conflict')
@@ -57,9 +55,9 @@ export default {
         : __('ui_refresh')
     },
     syncIcon() {
-      return !this.online ? 'offline_bolt'
-        : !this.hasToken ? 'cloud_off'
-        // : this.uploadSuccess ? 'cloud_done'
+      return !this.online ? 'cloud_off' // eslint-disable-line
+        : !this.hasToken ? 'cloud_off' // eslint-disable-line
+        : this.uploadSuccess ? 'cloud_done'
         : 'cloud_upload'
     },
   },
@@ -72,16 +70,15 @@ export default {
     ...mapMutations(['setConflict']),
     init() {
       this.onScroll()
-      window.addEventListener('online', () => this.online = true)
-      window.addEventListener('offline', () => this.online = false)
+      window.addEventListener('online', () => { this.online = true })
+      window.addEventListener('offline', () => { this.online = false })
       chrome.runtime.onMessage.addListener(msg => {
-        console.log(msg)
         if (msg.refresh) {
           this.syncing = true
         } else if (msg.refreshed) {
           this.syncing = false
           this.uploadSuccess = true
-          setTimeout(() => this.uploadSuccess = false, 3000)
+          setTimeout(() => { this.uploadSuccess = false }, 3000)
         }
       })
     },

src/page/main/DetailList.vue

@@ -139,7 +139,7 @@
               </v-list-tile-sub-title>
             </v-list-tile-content>
           </v-list-tile>
-          <v-layout v-if="list.tabs.length > 9 && !list.showAll">
+          <v-layout v-if="list.tabs.length > 10 && !list.showAll">
             <v-flex class="text-xs-center">
               <v-btn small flat @click="showAll(listIndex)"><v-icon>more_horiz</v-icon></v-btn>
             </v-flex>