From 462e85dad03c0ea5b28a2646b9db6c36fd4af289 Mon Sep 17 00:00:00 2001
From: QiroNT <i@ntz.im>
Date: Sat, 10 Dec 2022 05:18:21 +0800
Subject: [PATCH] refactor(command): use dialog (#352)

---
 app.vue                             |   1 +
 components/command/CommandPanel.vue | 203 +++++++++++-----------------
 components/command/CommandRoot.vue  |   7 -
 components/modal/ModalContainer.vue |  15 ++
 composables/command.ts              |   2 +-
 composables/dialog.ts               |  12 ++
 layouts/default.vue                 |   1 -
 7 files changed, 110 insertions(+), 131 deletions(-)
 delete mode 100644 components/command/CommandRoot.vue

diff --git a/app.vue b/app.vue
index d8319048..d2ba216a 100644
--- a/app.vue
+++ b/app.vue
@@ -1,6 +1,7 @@
 <script setup lang="ts">
 setupPageHeader()
 await setupI18n()
+provideGlobalCommands()
 
 // We want to trigger rerendering the page when account changes
 const key = computed(() => `${currentServer.value}:${currentUser.value?.account.id || ''}`)
diff --git a/components/command/CommandPanel.vue b/components/command/CommandPanel.vue
index 7a049016..01b79a39 100644
--- a/components/command/CommandPanel.vue
+++ b/components/command/CommandPanel.vue
@@ -1,31 +1,21 @@
 <script setup lang="ts">
 import type { CommandScope, QueryIndexedCommand } from '@/composables/command'
 
-const isMac = useIsMac()
+const emit = defineEmits<{
+  (event: 'close'): void
+}>()
+
 const registry = useCommandRegistry()
 
 const inputEl = $ref<HTMLInputElement>()
 const resultEl = $ref<HTMLDivElement>()
 
-let show = $ref(false)
-let scopes = $ref<CommandScope[]>([])
-let input = $ref('')
+const scopes = $ref<CommandScope[]>([])
+let input = $(commandPanelInput)
 
-// listen to ctrl+/ on windows/linux or cmd+/ on mac
-useEventListener('keydown', async (e: KeyboardEvent) => {
-  if (e.key === '/' && (isMac.value ? e.metaKey : e.ctrlKey)) {
-    e.preventDefault()
-    show = true
-    scopes = []
-    input = '>'
-    await nextTick()
-    inputEl?.focus()
-  }
+onMounted(() => {
+  inputEl?.focus()
 })
-onKeyStroke('Escape', (e) => {
-  e.preventDefault()
-  show = false
-}, { target: document })
 
 const commandMode = $computed(() => input.startsWith('>'))
 const result = $computed(() => commandMode
@@ -42,7 +32,7 @@ const findItemEl = (index: number) =>
 const onCommandActivate = (item: QueryIndexedCommand) => {
   if (item.onActivate) {
     item.onActivate()
-    show = false
+    emit('close')
   }
   else if (item.onComplete) {
     scopes.push(item.onComplete())
@@ -137,117 +127,86 @@ const onKeyDown = (e: KeyboardEvent) => {
 </script>
 
 <template>
-  <!-- Overlay -->
-  <Transition
-    enter-active-class="transition duration-200 ease-out"
-    enter-from-class="transform opacity-0"
-    enter-to-class="transform opacity-100"
-    leave-active-class="transition duration-100 ease-in"
-    leave-from-class="transform opacity-100"
-    leave-to-class="transform opacity-0"
-  >
-    <div
-      v-if="show"
-      class="z-100 fixed inset-0 opacity-70 bg-base"
-      @click="show = false"
-    />
-  </Transition>
+  <div class="flex flex-col w-50vw max-w-180 h-50vh max-h-120">
+    <!-- Input -->
+    <label class="flex mx-3 my-1 items-center">
+      <div mx-1 i-ri:search-line />
 
-  <!-- Panel -->
-  <Transition
-    enter-active-class="transition duration-65 ease-out"
-    enter-from-class="transform scale-95"
-    enter-to-class="transform scale-100"
-    leave-active-class="transition duration-100 ease-in"
-    leave-from-class="transform scale-100"
-    leave-to-class="transform scale-95"
-  >
-    <div v-if="show" class="z-100 fixed inset-0 grid place-items-center pointer-events-none">
-      <div
-        class="flex flex-col w-50vw h-50vh rounded-md bg-base shadow-lg pointer-events-auto"
-        border="1 base"
+      <div v-for="scope in scopes" :key="scope.id" class="flex items-center mx-1 gap-2">
+        <div class="text-sm">{{ scope.display }}</div>
+        <span class="text-secondary">/</span>
+      </div>
+
+      <input
+        ref="inputEl"
+        v-model="input"
+        class="focus:outline-none flex-1 p-2 rounded bg-base"
+        placeholder="Search"
+        @keydown="onKeyDown"
       >
-        <!-- Input -->
-        <label class="flex mx-3 my-1 items-center">
-          <div mx-1 i-ri:search-line />
 
-          <div v-for="scope in scopes" :key="scope.id" class="flex items-center mx-1 gap-2">
-            <div class="text-sm">{{ scope.display }}</div>
-            <span class="text-secondary">/</span>
-          </div>
+      <CommandKey name="Escape" />
+    </label>
 
-          <input
-            ref="inputEl"
-            v-model="input"
-            class="focus:outline-none flex-1 p-2 rounded bg-base"
-            placeholder="Search"
-            @keydown="onKeyDown"
+    <div class="w-full border-b-1 border-base" />
+
+    <!-- Results -->
+    <div ref="resultEl" class="flex-1 mx-1 overflow-y-auto">
+      <template v-for="[scope, group] in result.grouped" :key="scope">
+        <div class="mt-2 px-2 py-1 text-sm text-secondary">
+          {{ scope }}
+        </div>
+
+        <template v-for="cmd in group" :key="cmd.index">
+          <div
+            class="flex px-3 py-2 my-1 items-center rounded-lg hover:bg-active transition-all duration-65 ease-in-out cursor-pointer scroll-m-10"
+            :class="{ 'bg-active': active === cmd.index }"
+            :data-index="cmd.index"
+            @click="onCommandActivate(cmd)"
           >
+            <div v-if="cmd.icon" mr-2 :class="cmd.icon" />
 
-          <CommandKey name="Escape" />
-        </label>
-
-        <div class="w-full border-b-1 border-base" />
-
-        <!-- Results -->
-        <div ref="resultEl" class="flex-1 mx-1 overflow-y-auto">
-          <template v-for="[scope, group] in result.grouped" :key="scope">
-            <div class="mt-2 px-2 py-1 text-sm text-secondary">
-              {{ scope }}
+            <div class="flex-1 flex items-baseline gap-2">
+              <div :class="{ 'font-medium': active === cmd.index }">
+                {{ cmd.name }}
+              </div>
+              <div v-if="cmd.description" class="text-xs text-secondary">
+                {{ cmd.description }}
+              </div>
             </div>
 
-            <template v-for="cmd in group" :key="cmd.index">
-              <div
-                class="flex px-3 py-2 my-1 items-center rounded-lg hover:bg-active transition-all duration-65 ease-in-out cursor-pointer scroll-m-10"
-                :class="{ 'bg-active': active === cmd.index }"
-                :data-index="cmd.index"
-                @click="onCommandActivate(cmd)"
-              >
-                <div v-if="cmd.icon" mr-2 :class="cmd.icon" />
-
-                <div class="flex-1 flex items-baseline gap-2">
-                  <div :class="{ 'font-medium': active === cmd.index }">
-                    {{ cmd.name }}
-                  </div>
-                  <div v-if="cmd.description" class="text-xs text-secondary">
-                    {{ cmd.description }}
-                  </div>
-                </div>
-
-                <div
-                  v-if="cmd.onComplete"
-                  class="flex items-center gap-1 transition-all duration-65 ease-in-out"
-                  :class="active === cmd.index ? 'opacity-100' : 'opacity-0'"
-                >
-                  <div class="text-xs text-secondary">
-                    {{ $t('command.complete') }}
-                  </div>
-                  <CommandKey name="Tab" />
-                </div>
-                <div
-                  v-if="cmd.onActivate"
-                  class="flex items-center gap-1 transition-all duration-65 ease-in-out"
-                  :class="active === cmd.index ? 'opacity-100' : 'opacity-0'"
-                >
-                  <div class="text-xs text-secondary">
-                    {{ $t('command.activate') }}
-                  </div>
-                  <CommandKey name="Enter" />
-                </div>
+            <div
+              v-if="cmd.onComplete"
+              class="flex items-center gap-1 transition-all duration-65 ease-in-out"
+              :class="active === cmd.index ? 'opacity-100' : 'opacity-0'"
+            >
+              <div class="text-xs text-secondary">
+                {{ $t('command.complete') }}
               </div>
-            </template>
-          </template>
-        </div>
-
-        <div class="w-full border-b-1 border-base" />
-
-        <!-- Footer -->
-        <div class="flex items-center px-3 py-1 text-xs">
-          <div i-ri:lightbulb-flash-line /> Tip: Use
-          <!-- <CommandKey name="Ctrl+K" /> to search, -->
-          <CommandKey name="Ctrl+/" /> to activate command mode.
-        </div>
-      </div>
+              <CommandKey name="Tab" />
+            </div>
+            <div
+              v-if="cmd.onActivate"
+              class="flex items-center gap-1 transition-all duration-65 ease-in-out"
+              :class="active === cmd.index ? 'opacity-100' : 'opacity-0'"
+            >
+              <div class="text-xs text-secondary">
+                {{ $t('command.activate') }}
+              </div>
+              <CommandKey name="Enter" />
+            </div>
+          </div>
+        </template>
+      </template>
     </div>
-  </Transition>
+
+    <div class="w-full border-b-1 border-base" />
+
+    <!-- Footer -->
+    <div class="flex items-center px-3 py-1 text-xs">
+      <div i-ri:lightbulb-flash-line /> Tip: Use
+      <!-- <CommandKey name="Ctrl+K" /> to search, -->
+      <CommandKey name="Ctrl+/" /> to activate command mode.
+    </div>
+  </div>
 </template>
diff --git a/components/command/CommandRoot.vue b/components/command/CommandRoot.vue
deleted file mode 100644
index 79a660b7..00000000
--- a/components/command/CommandRoot.vue
+++ /dev/null
@@ -1,7 +0,0 @@
-<script setup lang="ts">
-provideGlobalCommands()
-</script>
-
-<template>
-  <CommandPanel />
-</template>
diff --git a/components/modal/ModalContainer.vue b/components/modal/ModalContainer.vue
index 24f179c4..56e3dfb2 100644
--- a/components/modal/ModalContainer.vue
+++ b/components/modal/ModalContainer.vue
@@ -1,11 +1,23 @@
 <script setup lang="ts">
 import {
+  isCommandPanelOpen,
   isEditHistoryDialogOpen,
   isMediaPreviewOpen,
   isPreviewHelpOpen,
   isPublishDialogOpen,
   isSigninDialogOpen,
 } from '~/composables/dialog'
+
+const isMac = useIsMac()
+
+// TODO: temporary, await for keybind system
+// listen to ctrl+/ on windows/linux or cmd+/ on mac
+useEventListener('keydown', (e: KeyboardEvent) => {
+  if (e.key === '/' && (isMac.value ? e.metaKey : e.ctrlKey)) {
+    e.preventDefault()
+    openCommandPanel(true)
+  }
+})
 </script>
 
 <template>
@@ -30,4 +42,7 @@ import {
   <ModalDialog v-model="isEditHistoryDialogOpen">
     <StatusEditPreview :edit="statusEdit" />
   </ModalDialog>
+  <ModalDialog v-model="isCommandPanelOpen" max-w-fit flex>
+    <CommandPanel @close="closeCommandPanel()" />
+  </ModalDialog>
 </template>
diff --git a/composables/command.ts b/composables/command.ts
index 44cfe24b..34644ed6 100644
--- a/composables/command.ts
+++ b/composables/command.ts
@@ -281,7 +281,7 @@ export const provideGlobalCommands = () => {
     visible: () => users.value.length > 1,
 
     name: () => t('action.switch_account'),
-    description: t('command.switch_account_desc'),
+    description: () => t('command.switch_account_desc'),
     icon: 'i-ri:user-shared-line',
 
     onComplete: () => ({
diff --git a/composables/dialog.ts b/composables/dialog.ts
index 9dbe5e82..0afd2df5 100644
--- a/composables/dialog.ts
+++ b/composables/dialog.ts
@@ -8,6 +8,8 @@ export const mediaPreviewIndex = ref(0)
 export const statusEdit = ref<StatusEdit>()
 export const dialogDraftKey = ref<string>()
 
+export const commandPanelInput = ref('')
+
 export const isFirstVisit = useLocalStorage(STORAGE_KEY_FIRST_VISIT, !process.mock)
 export const isZenMode = useLocalStorage(STORAGE_KEY_ZEN_MODE, false)
 
@@ -16,6 +18,7 @@ export const isPublishDialogOpen = ref(false)
 export const isMediaPreviewOpen = ref(false)
 export const isEditHistoryDialogOpen = ref(false)
 export const isPreviewHelpOpen = ref(isFirstVisit.value)
+export const isCommandPanelOpen = ref(false)
 
 export const toggleZenMode = useToggle(isZenMode)
 
@@ -72,3 +75,12 @@ export function openPreviewHelp() {
 export function closePreviewHelp() {
   isPreviewHelpOpen.value = false
 }
+
+export function openCommandPanel(isCommandMode = false) {
+  commandPanelInput.value = isCommandMode ? '>' : ''
+  isCommandPanelOpen.value = true
+}
+
+export function closeCommandPanel() {
+  isCommandPanelOpen.value = false
+}
diff --git a/layouts/default.vue b/layouts/default.vue
index 8a9b887f..0c62c1e7 100644
--- a/layouts/default.vue
+++ b/layouts/default.vue
@@ -51,6 +51,5 @@
       </aside>
     </main>
     <ModalContainer />
-    <CommandRoot />
   </div>
 </template>