perf: disable load value when right-click in tree view

frontend/src/components/sidebar/BrowserTree.vue

@@ -231,29 +231,11 @@ const handleSelectContextMenu = (key) => {
 }
 
 const onUpdateSelectedKeys = (keys, options) => {
-    try {
+    if (!isEmpty(keys)) {
-        if (!isEmpty(options)) {
+        tabStore.setSelectedKeys(props.server, keys)
-            // prevent load duplicate key
+    } else {
-            for (const node of options) {
-                if (node.type === ConnectionType.RedisValue) {
-                    const { key, db } = node
-                    const redisKey = node.redisKeyCode || node.redisKey
-                    if (!includes(selectedKeys.value, key)) {
-                        browserStore.loadKeySummary({
-                            server: props.server,
-                            db,
-                            key: redisKey,
-                            clearValue: true,
-                        })
-                    }
-                    return
-                }
-            }
-        }
         // default is load blank key to display server status
         // tabStore.openBlank(props.server)
-    } finally {
-        tabStore.setSelectedKeys(props.server, keys)
     }
 }
 
@@ -485,6 +467,19 @@ const renderSuffix = ({ option }) => {
 
 const nodeProps = ({ option }) => {
     return {
+        onClick: () => {
+            if (option.type === ConnectionType.RedisValue) {
+                if (tabStore.setActivatedKey(props.server, option.key)) {
+                    const { db, redisKey, redisKeyCode } = option
+                    browserStore.loadKeySummary({
+                        server: props.server,
+                        db,
+                        key: redisKeyCode || redisKey,
+                        clearValue: true,
+                    })
+                }
+            }
+        },
         onDblclick: () => {
             if (props.loading) {
                 console.warn('TODO: alert to ignore double click when loading')
@@ -506,7 +501,7 @@ const nodeProps = ({ option }) => {
                 contextMenuParam.x = e.clientX
                 contextMenuParam.y = e.clientY
                 contextMenuParam.show = true
-                onUpdateSelectedKeys([option.key], [option])
+                // onUpdateSelectedKeys([option.key], [option])
             })
         },
         // onMouseover() {
@@ -572,7 +567,7 @@ defineExpose({
 </script>
 
 <template>
-    <div :style="{ backgroundColor }" class="flex-box-v browser-tree-wrapper">
+    <div :style="{ backgroundColor }" class="flex-box-v browser-tree-wrapper" @contextmenu="(e) => e.preventDefault()">
         <n-spin v-if="props.loading" class="fill-height" />
         <n-empty v-else-if="!props.loading && isEmpty(data)" class="empty-content" />
         <n-tree

frontend/src/objects/tabItem.js

@@ -16,6 +16,7 @@ export class TabItem {
      * @param {string} subTab secondary tab value
      * @param {string} [title] tab title
      * @param {string} [icon] tab icon
+     * @param {string} [activatedKey] current activated key on displaying
      * @param {string[]} expandedKeys
      * @param {string[]} selectedKeys
      * @param {CheckedKey[]} checkedKeys
@@ -63,6 +64,7 @@ export class TabItem {
         this.blank = blank
         this.subTab = subTab
         this.icon = icon
+        this.activatedKey = ''
         this.expandedKeys = expandedKeys
         this.selectedKeys = selectedKeys
         this.checkedKeys = checkedKeys

frontend/src/stores/tab.js

@@ -101,11 +101,6 @@ const useTabStore = defineStore('tab', {
             return get(this.tabs, [this.activatedIndex, 'name'])
         },
 
-        currentSelectedKeys() {
-            const tab = this.currentTab
-            return get(tab, 'selectedKeys', [])
-        },
-
         currentCheckedKeys() {
             const tab = this.currentTab
             return get(tab, 'checkedKeys', [])
@@ -748,6 +743,23 @@ const useTabStore = defineStore('tab', {
                 }
             }
         },
+
+        /**
+         * set activated key
+         * @param {string} server
+         * @param {string} key
+         */
+        setActivatedKey(server, key) {
+            /** @type TabItem**/
+            let tab = find(this.tabList, { name: server })
+            if (tab != null) {
+                if (!isEmpty(key) && key !== tab.activatedKey) {
+                    tab.activatedKey = key
+                    return true
+                }
+            }
+            return false
+        },
     },
 })