From d67c88af16555f2e82cfc3b24de4628a4742709c Mon Sep 17 00:00:00 2001
From: maropboia <164220066+maropboia@users.noreply.github.com>
Date: Fri, 3 May 2024 11:38:54 +0600
Subject: [PATCH] improve

---
 .vitepress/theme/components/Modal.vue | 67 +++++++++++++++++++++------
 1 file changed, 52 insertions(+), 15 deletions(-)

diff --git a/.vitepress/theme/components/Modal.vue b/.vitepress/theme/components/Modal.vue
index 349d6745e..b20e55660 100644
--- a/.vitepress/theme/components/Modal.vue
+++ b/.vitepress/theme/components/Modal.vue
@@ -3,21 +3,50 @@ import { ref } from 'vue'
 import Feedback from './Feedback.vue'
 
 const showModal = ref(false)
+
+const handleButtonClick = () => {
+  showModal.value = true
+}
+
+const handleButtonKeydown = (event: KeyboardEvent) => {
+  if (event.key === 'Enter') {
+    handleButtonClick()
+  }
+}
+
+const handleModalMaskClick = () => {
+  showModal.value = false
+}
+
+const handleModalKeydown = (event: KeyboardEvent) => {
+  if (event.key === 'Escape') {
+    showModal.value = false
+  }
+}
 </script>
 
 <template>
   <button
     class="p-[4px 8px] text-xl i-carbon:user-favorite-alt-filled"
-    @click="showModal = true"
+    type="button"
+    aria-haspopup="dialog"
+    aria-expanded="false"
+    @click="handleButtonClick"
+    @keydown="handleButtonKeydown"
   />
 
   <Teleport to="body">
     <Transition name="modal">
-      <div v-show="showModal" class="modal-mask">
+      <div
+        v-show="showModal"
+        class="modal-mask"
+        @mousedown="handleModalMaskClick"
+        @keydown="handleModalKeydown"
+      >
         <div class="modal-container">
           <Feedback />
           <div class="model-footer">
-            <button class="modal-button" @click="showModal = false">
+            <button class="modal-button" @click="showModal = false" aria-label="Close">
               Close
             </button>
           </div>
@@ -39,7 +68,7 @@ const showModal = ref(false)
   display: flex;
   align-items: center;
   justify-content: center;
-  transition: opacity 0.3s ease;
+  transition: opacity 0.3s ease, transform 0.3s ease;
 }
 
 .modal-container {
@@ -49,7 +78,25 @@ const showModal = ref(false)
   background-color: var(--vp-c-bg);
   border-radius: 2px;
   box-shadow: 0 2px 8px rgba(0, 0, 0, 0.33);
-  transition: all 0.3s ease;
+  opacity: 0;
+  transform: scale(1.1);
+  transition: opacity 0.3s ease, transform 0.3s ease 0.1s;
+}
+
+.modal-enter-active,
+.modal-leave-active {
+  transition: opacity 0.3s ease, transform 0.3s ease;
+}
+
+.modal-enter-from,
+.modal-leave-to {
+  opacity: 0;
+  transform: scale(1.1);
+}
+
+.modal-container.modal-enter-active,
+.modal-container.modal-leave-active {
+  transition: opacity 0.3s ease, transform 0.3s ease 0.1s;
 }
 
 .model-footer {
@@ -70,14 +117,4 @@ const showModal = ref(false)
   color: var(--vp-button-alt-hover-text);
   background-color: var(--vp-button-alt-hover-bg);
 }
-
-.modal-enter-from,
-.modal-leave-to {
-  opacity: 0;
-}
-
-.modal-enter-from .modal-container,
-.modal-leave-to .modal-container {
-  transform: scale(1.1);
-}
 </style>