select fixes (#1057)

* select fixes

* changeset

---------

Co-authored-by: thejackshelton <me@jackshelton.com>

Author: thejackshelton-kunaico

.changeset/popular-cups-study.md

@@ -0,0 +1,5 @@
+---
+'@qwik-ui/headless': patch
+---
+
+fix: select uses correct types and does not execute focus on server side

packages/kit-headless/src/components/select/select-inline.tsx

@@ -85,7 +85,11 @@ export const HSelectRoot: Component<SelectProps & InlineCompProps> = (
         // distinct value, or the display value is the same as the value
         const value = (givenItemValue !== null ? givenItemValue : displayValue) as string;
 
-        itemsMap.set(currItemIndex, { value, displayValue, disabled: isItemDisabled });
+        itemsMap.set(currItemIndex, {
+          value,
+          displayValue,
+          disabled: isItemDisabled,
+        });
 
         if (props.value && props.multiple) {
           throw new Error(
@@ -99,7 +103,9 @@ export const HSelectRoot: Component<SelectProps & InlineCompProps> = (
           valuePropIndex = currItemIndex;
         }
 
-        const isString = typeof child.props.children === 'string';
+        const isString =
+          typeof child.props.children === 'string' ||
+          typeof child.props.children === 'object';
 
         if (!isString) {
           throw new Error(

packages/kit-headless/src/components/select/select-item.tsx

@@ -61,7 +61,9 @@ export const HSelectItem = component$<SelectItemProps>((props) => {
     if (disabled) return;
 
     if (context.highlightedIndexSig.value === _index) {
-      itemRef.value?.focus({ preventScroll: true });
+      if (!isServer) {
+        itemRef.value?.focus({ preventScroll: true });
+      }
       return true;
     } else {
       return false;