From d17352b8f36c623c663c7293e7237cd5592b312a Mon Sep 17 00:00:00 2001
From: 18alantom <2.alan.tom@gmail.com>
Date: Mon, 5 Dec 2022 16:44:34 +0530
Subject: [PATCH] fix: toggle popover on click - hide clear if required

---
 src/components/Controls/Datetime.vue       | 10 +++-------
 src/components/Controls/DatetimePicker.vue | 12 +++++++-----
 2 files changed, 10 insertions(+), 12 deletions(-)

diff --git a/src/components/Controls/Datetime.vue b/src/components/Controls/Datetime.vue
index 0ddace70..a3600b25 100644
--- a/src/components/Controls/Datetime.vue
+++ b/src/components/Controls/Datetime.vue
@@ -33,17 +33,13 @@
     </template>
 
     <!-- Datetime Input Popover -->
-    <template #content="{ togglePopover }">
+    <template #content>
       <DatetimePicker
+        :show-clear="!isRequired"
         :select-time="selectTime"
         :model-value="internalValue"
         :format-value="formatValue"
-        @update:model-value="
-          (value) => {
-            triggerChange(value);
-            togglePopover();
-          }
-        "
+        @update:model-value="(value) => triggerChange(value)"
       />
     </template>
   </Popover>
diff --git a/src/components/Controls/DatetimePicker.vue b/src/components/Controls/DatetimePicker.vue
index aa08b118..8d3c9d98 100644
--- a/src/components/Controls/DatetimePicker.vue
+++ b/src/components/Controls/DatetimePicker.vue
@@ -2,10 +2,7 @@
   <div>
     <!-- Datetime header -->
     <div class="flex justify-between items-center text-sm px-4 pt-4">
-      <div
-        class="text-blue-500 cursor-pointer"
-        @click="selectMonthYear = !selectMonthYear"
-      >
+      <div class="text-blue-500">
         {{ datetimeString }}
       </div>
 
@@ -88,7 +85,10 @@
         class="border-l flex flex-col justify-between"
       >
         <!-- Month Selector -->
-        <div class="flex flex-col gap-2 overflow-auto h-40 p-4">
+        <div
+          class="flex flex-col gap-2 overflow-auto m-4"
+          style="height: calc(248px - 79.5px - 1px - 2rem)"
+        >
           <div
             v-for="(m, i) of months"
             :key="m"
@@ -170,6 +170,7 @@
       </button>
 
       <button
+        v-if="showClear"
         class="text-xs text-gray-600 hover:text-gray-600 ml-auto"
         @click="clearClicked"
       >
@@ -204,6 +205,7 @@ export default defineComponent({
   props: {
     modelValue: { type: Date },
     selectTime: { type: Boolean, default: true },
+    showClear: { type: Boolean, default: true },
     formatValue: { type: Function as PropType<(value: Date | null) => string> },
   },
   mounted() {