docs: fix private trapFocus prop being expose/generated

unovue · Jan 21, 2025 · 6b5af16 · 6b5af16
1 parent f0127c6
commit 6b5af16
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 22 deletions.
diff --git a/docs/content/meta/DatePickerContent.md b/docs/content/meta/DatePickerContent.md
@@ -110,12 +110,6 @@
     'type': '\'partial\' | \'always\'',
     'required': false
   },
-  {
-    'name': 'trapFocus',
-    'description': '<p>Whether focus should be trapped within the <code>MenuContent</code></p>\n',
-    'type': 'boolean',
-    'required': false
-  },
   {
     'name': 'updatePositionStrategy',
     'description': '<p>Strategy to update the position of the floating element on every animation frame.</p>\n',

diff --git a/docs/content/meta/DateRangePickerContent.md b/docs/content/meta/DateRangePickerContent.md
@@ -110,12 +110,6 @@
     'type': '\'partial\' | \'always\'',
     'required': false
   },
-  {
-    'name': 'trapFocus',
-    'description': '<p>Whether focus should be trapped within the <code>MenuContent</code></p>\n',
-    'type': 'boolean',
-    'required': false
-  },
   {
     'name': 'updatePositionStrategy',
     'description': '<p>Strategy to update the position of the floating element on every animation frame.</p>\n',

diff --git a/docs/content/meta/PopoverContent.md b/docs/content/meta/PopoverContent.md
@@ -110,12 +110,6 @@
     'type': '\'partial\' | \'always\'',
     'required': false
   },
-  {
-    'name': 'trapFocus',
-    'description': '<p>Whether focus should be trapped within the <code>MenuContent</code></p>\n',
-    'type': 'boolean',
-    'required': false
-  },
   {
     'name': 'updatePositionStrategy',
     'description': '<p>Strategy to update the position of the floating element on every animation frame.</p>\n',

diff --git a/packages/core/src/Popover/PopoverContentImpl.vue b/packages/core/src/Popover/PopoverContentImpl.vue
@@ -20,9 +20,9 @@ export type PopoverContentImplEmits = DismissableLayerEmits & {
   closeAutoFocus: [event: Event]
 }
 
-export interface PopoverContentImplProps
-  extends PopperContentProps,
-  DismissableLayerProps {
+export interface PopoverContentImplProps extends PopperContentProps, DismissableLayerProps {}
+
+interface PopoverContentImplPrivateProps extends PopoverContentImplProps {
   /**
    * Whether focus should be trapped within the `MenuContent`
    * @defaultValue false
@@ -38,7 +38,7 @@ import { DismissableLayer } from '@/DismissableLayer'
 import { FocusScope } from '@/FocusScope'
 import { useFocusGuards, useForwardExpose, useForwardProps } from '@/shared'
 
-const props = defineProps<PopoverContentImplProps>()
+const props = defineProps<PopoverContentImplPrivateProps>()
 const emits = defineEmits<PopoverContentImplEmits>()
 
 const forwarded = useForwardProps(reactiveOmit(props, 'trapFocus', 'disableOutsidePointerEvents'))