Merge pull request #155 from PrestaShopCorp/feat/menu-slot-props

[Feature]: add menu open state to slot props
PrestaShopCorp · Jul 7, 2023 · 95054c2 · 95054c2
2 parents 7c81dc8 + 942afa7
commit 95054c2
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 2 deletions.
diff --git a/packages/components/menu/src/menu.vue b/packages/components/menu/src/menu.vue
@@ -1,11 +1,12 @@
 <template>
   <Popover
+    v-slot="{ open }"
     class="puik-menu"
     :class="[`puik-menu--position-${position}`, `puik-menu--align-${align}`]"
     as="div"
   >
     <PopoverButton class="puik-menu__trigger" as="template">
-      <slot name="trigger"></slot>
+      <slot name="trigger" :open="open"></slot>
     </PopoverButton>
 
     <transition
@@ -21,7 +22,7 @@
         class="puik-menu__content"
         :style="{ maxHeight, width }"
       >
-        <slot :close="close"></slot>
+        <slot :close="close" :open="open"></slot>
       </PopoverPanel>
     </transition>
   </Popover>

diff --git a/packages/components/menu/stories/menu.stories.ts b/packages/components/menu/stories/menu.stories.ts
@@ -61,10 +61,31 @@ export default {
     trigger: {
       control: 'none',
       description: 'Trigger used to show or hide menu',
+      table: {
+        type: {
+          summary: 'SlotProps',
+          detail: `
+  {
+    open: boolean
+  }
+          `,
+        },
+      },
     },
     default: {
       control: 'none',
       description: 'Menu content',
+      table: {
+        type: {
+          summary: 'SlotProps',
+          detail: `
+  {
+    open: boolean,
+    close: () => void
+  }
+            `,
+        },
+      },
     },
   },
   args: {