fix(popover): simplify Popover type to make ts happy

Author: maiieul

packages/kit-headless/src/components/popover/popover.tsx

@@ -4,29 +4,29 @@ import { PopoverImpl } from './popover-impl';
 import { PopoverImplProps } from './popover-impl';
 import { FloatingProps } from './floating';
 
-type PopoverProps = PopoverImplProps & ({ floating?: true } & FloatingProps);
+// TODO: improve the type so that it only includes FloatingProps when floating is true.
+type PopoverProps = PopoverImplProps & { floating?: boolean } & FloatingProps;
 
 /* This component determines whether the popover needs floating behavior, a common example where it doesn't, would be a toast. */
-export const Popover = component$<PopoverProps>(
-  ({ floating, anchorRef, ref, ...props }) => {
-    if (floating) {
-      if (!anchorRef) {
-        throw new Error(
-          'Qwik UI Popover: anchorRef is required on the popover when floating is true',
-        );
-      }
-
-      return (
-        <FloatingPopover ref={ref} anchorRef={anchorRef} {...props}>
-          <Slot />
-        </FloatingPopover>
+export const Popover = component$<PopoverProps>((props) => {
+  if (props.floating) {
+    const { ref, anchorRef, ...rest } = props;
+    if (!anchorRef) {
+      throw new Error(
+        'Qwik UI Popover: anchorRef is required on the popover when floating is true',
       );
     }
 
     return (
-      <PopoverImpl {...props}>
+      <FloatingPopover ref={ref} anchorRef={anchorRef} {...rest}>
         <Slot />
-      </PopoverImpl>
+      </FloatingPopover>
     );
-  },
-);
+  }
+
+  return (
+    <PopoverImpl {...props}>
+      <Slot />
+    </PopoverImpl>
+  );
+});