@radix-ui/react-select 1.0.1-rc.7 → 1.0.1-rc.9

package/dist/index.module.js

@@ -11,7 +11,6 @@ import {DismissableLayer as $01b9c$DismissableLayer} from "@radix-ui/react-dismi
 import {useFocusGuards as $01b9c$useFocusGuards} from "@radix-ui/react-focus-guards";
 import {FocusScope as $01b9c$FocusScope} from "@radix-ui/react-focus-scope";
 import {useId as $01b9c$useId} from "@radix-ui/react-id";
-import {useLabelContext as $01b9c$useLabelContext} from "@radix-ui/react-label";
 import {Portal as $01b9c$Portal} from "@radix-ui/react-portal";
 import {Primitive as $01b9c$Primitive} from "@radix-ui/react-primitive";
 import {Slot as $01b9c$Slot} from "@radix-ui/react-slot";
@@ -44,7 +43,6 @@ import {RemoveScroll as $01b9c$RemoveScroll} from "react-remove-scroll";
 
 
 
-
 
 
 const $cc7e05a45900e73f$var$OPEN_KEYS = [
@@ -139,12 +137,10 @@ const $cc7e05a45900e73f$export$ef9b1a59e592288f = (props)=>{
  * SelectTrigger
  * -----------------------------------------------------------------------------------------------*/ const $cc7e05a45900e73f$var$TRIGGER_NAME = 'SelectTrigger';
 const $cc7e05a45900e73f$export$3ac1e88a1c0b9f1 = /*#__PURE__*/ $01b9c$forwardRef((props, forwardedRef)=>{
-    const { __scopeSelect: __scopeSelect , disabled: disabled = false , 'aria-labelledby': ariaLabelledby , ...triggerProps } = props;
+    const { __scopeSelect: __scopeSelect , disabled: disabled = false , ...triggerProps } = props;
     const context = $cc7e05a45900e73f$var$useSelectContext($cc7e05a45900e73f$var$TRIGGER_NAME, __scopeSelect);
     const composedRefs = $01b9c$useComposedRefs(forwardedRef, context.onTriggerChange);
     const getItems = $cc7e05a45900e73f$var$useCollection(__scopeSelect);
-    const labelId = $01b9c$useLabelContext(context.trigger);
-    const labelledBy = ariaLabelledby || labelId;
     const [searchRef, handleTypeaheadSearch, resetTypeahead] = $cc7e05a45900e73f$var$useTypeaheadSearch((search)=>{
         const enabledItems = getItems().filter((item)=>!item.disabled
         );
@@ -165,14 +161,22 @@ const $cc7e05a45900e73f$export$3ac1e88a1c0b9f1 = /*#__PURE__*/ $01b9c$forwardRef
         "aria-controls": context.contentId,
         "aria-expanded": context.open,
         "aria-autocomplete": "none",
-        "aria-labelledby": labelledBy,
         dir: context.dir,
         "data-state": context.open ? 'open' : 'closed',
         disabled: disabled,
         "data-disabled": disabled ? '' : undefined,
         "data-placeholder": context.value === undefined ? '' : undefined
     }, triggerProps, {
-        ref: composedRefs,
+        ref: composedRefs // Enable compatibility with native label or custom `Label` "click" for Safari:
+        ,
+        onClick: $01b9c$composeEventHandlers(triggerProps.onClick, (event)=>{
+            // Whilst browsers generally have no issue focusing the trigger when clicking
+            // on a label, Safari seems to struggle with the fact that there's no `onClick`.
+            // We force `focus` in this case. Note: this doesn't create any other side-effect
+            // because we are preventing default in `onPointerDown` so effectively
+            // this only runs for a label "click"
+            event.currentTarget.focus();
+        }),
         onPointerDown: $01b9c$composeEventHandlers(triggerProps.onPointerDown, (event)=>{
             // prevent implicit pointer capture
             // https://www.w3.org/TR/pointerevents3/#implicit-pointer-capture