fix(dashboard): Remove unnecessary reason check from dismiss prevention

Drop eventDetails.reason checks from onOpenChange — as the maintainer
noted, onOpenChange is only called from Base UI's own interactions
(outside-press, escape-key, focus-out), not from our programmatic
setOpen(false) calls. Checking specific reasons would miss focus-out.

Author: Ryrahul

packages/dashboard/src/app/routes/_authenticated/_orders/components/customer-address-selector.tsx

@@ -97,13 +97,9 @@ export function CustomerAddressSelector({
         <Popover
             open={open}
             onOpenChange={(value, eventDetails) => {
-                if (
-                    !value &&
-                    effectiveTab === 'new' &&
-                    (eventDetails.reason === 'outside-press' || eventDetails.reason === 'escape-key')
-                ) {
-                    eventDetails.cancel();
-                    return;
+                // While the form tab is open, only our own submit/cancel may close the popover
+                if (!value && effectiveTab === 'new') {
+                    return eventDetails.cancel();
                 }
                 setOpen(value);
                 if (!value) {

packages/dashboard/src/lib/components/shared/customer-selector.tsx

@@ -234,13 +234,9 @@ export function CustomerSelector(props: CustomerSelectorProps) {
         <Popover
             open={open}
             onOpenChange={(isOpen, eventDetails) => {
-                if (
-                    !isOpen &&
-                    activeTab === 'new' &&
-                    (eventDetails.reason === 'outside-press' || eventDetails.reason === 'escape-key')
-                ) {
-                    eventDetails.cancel();
-                    return;
+                // While the create tab is open, only our own submit/cancel may close the popover
+                if (!isOpen && activeTab === 'new') {
+                    return eventDetails.cancel();
                 }
                 setOpen(isOpen);
                 if (!isOpen) {