@lotics/ui 2.3.0 → 2.3.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lotics/ui",
-  "version": "2.3.0",
+  "version": "2.3.1",
   "type": "module",
   "exports": {
     "./tokens": "./src/tokens.ts",

package/src/combobox.tsx

@@ -132,6 +132,13 @@ export function Combobox<T extends string = string, D = unknown>(props: Combobox
   const triggerRef = useRef<View>(null);
   const inputRef = useRef<RNTextInput>(null);
   const scrollRef = useRef<ScrollView>(null);
+  // Options are focusable (RN-Web Pressable), so a mouse-press on one blurs the
+  // input; commit then refocuses it to keep the textbox focus (aria pattern),
+  // which would re-fire onFocus and reopen the popover. This one-shot flag lets
+  // that programmatic refocus pass without reopening — set on a single-select
+  // commit, consumed by the next onFocus, and cleared on blur (the keyboard
+  // path never blurs, so the refocus is a no-op and onFocus never fires).
+  const suppressFocusOpenRef = useRef(false);
   const baseId = useId();
   const listboxId = `${baseId}-listbox`;
   const optionId = (i: number) => `${baseId}-option-${i}`;
@@ -188,6 +195,9 @@ export function Combobox<T extends string = string, D = unknown>(props: Combobox
         setQuery("");
       }
       setOpen(false);
+      // Single-select: the refocus below must not reopen the just-closed menu.
+      // Multi-select keeps the menu open to add more, so it isn't suppressed.
+      if (!multi) suppressFocusOpenRef.current = true;
       inputRef.current?.focus();
     },
     [onValueChange, isServer, debouncedSearch, onSearchChange, multi, reflectSelection],
@@ -278,8 +288,19 @@ export function Combobox<T extends string = string, D = unknown>(props: Combobox
           onClear={onClear}
           onChangeText={handleChangeText}
           onFocus={() => {
+            // A programmatic refocus right after a single-select commit must not
+            // reopen the menu; consume the one-shot flag instead of opening.
+            if (suppressFocusOpenRef.current) {
+              suppressFocusOpenRef.current = false;
+              return;
+            }
             if (!disabled) setOpen(true);
           }}
+          onBlur={() => {
+            // Clear a flag the keyboard path left set (its refocus was a no-op,
+            // so onFocus never consumed it) — the next real focus opens normally.
+            suppressFocusOpenRef.current = false;
+          }}
           onKeyPress={handleKeyPress}
           placeholder={chips.length > 0 ? undefined : placeholder}
           placeholderTextColor={colors.zinc["400"]}