fix: improve disabled state handling for Select.Trigger (#1134)

Author: Caesarovich

.changeset/eager-colts-clean.md

@@ -0,0 +1,5 @@
+---
+'@qwik-ui/headless': patch
+---
+
+Fix disabled state handling for Select.Trigger

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

@@ -3,6 +3,7 @@ import {
   Slot,
   component$,
   sync$,
+  useComputed$,
   useContext,
   useSignal,
   type PropsOf,
@@ -27,6 +28,10 @@ export const HSelectTrigger = component$<SelectTriggerProps>((props) => {
   const contextRefOpts = { context, givenContextRef: context.triggerRef };
   const triggerRef = useCombinedRef(props.ref, contextRefOpts);
 
+  const isDisabled = useComputed$(() =>
+    context.isDisabledSig.value || props.disabled ? true : undefined,
+  );
+
   const handleClickSync$ = sync$((e: MouseEvent) => {
     e.preventDefault();
   });
@@ -132,15 +137,15 @@ export const HSelectTrigger = component$<SelectTriggerProps>((props) => {
       onKeyDown$={[handleKeyDownSync$, handleKeyDown$, props.onKeyDown$]}
       data-open={context.isListboxOpenSig.value ? '' : undefined}
       data-closed={!context.isListboxOpenSig.value ? '' : undefined}
-      data-disabled={context.isDisabledSig.value ? '' : undefined}
+      data-disabled={isDisabled.value ? '' : undefined}
       data-invalid={context.isInvalidSig?.value ? '' : undefined}
       aria-labelledby={`${valueId} ${labelId} `}
       aria-describedby={`${descriptionId} 
       ${errorMessageId}`}
       aria-expanded={context.isListboxOpenSig.value ? true : false}
       aria-haspopup="listbox"
       aria-controls={panelId}
-      disabled={context.isDisabledSig.value ? true : undefined}
+      disabled={isDisabled.value}
       preventdefault:blur
     >
       <Slot />