Collection block bug fixes (#1852)

 - Navigate the row-actions relation icon to the relation's `entityId` rather than its `id`, matching LinkableRelationChip. The previous URL  pointed at a non-existent entity. Lookup is deferred until the popover opens, so list/gallery views (which mount the component per row) don't pay an O(rows × relations) scan on every render.
- Keep the row-actions popover open while its nested space selector is active. The selector's content is portaled outside the outer popover's DOM, so opening it fired `onFocusOutside` on the outer Content and unmounted both. Suppress the outer's dismiss handlers while the nested popover is open and close it explicitly on selection.
- `SelectSpaceAsPopover` gains optional controlled `open`/`onOpenChange` props (used by the row-actions wrapper above). The props are typed as a discriminated union so passing `open` without `onOpenChange` is a type error — otherwise Radix could end up stuck open with no updater.

Author: b-d055

apps/web/design-system/select-space-dialog.tsx

@@ -4,17 +4,31 @@ import * as React from 'react';
 
 import { SelectSpace } from './select-space';
 
-type SelectEntityAsPopoverProps = {
+type SelectEntityAsPopoverBaseProps = {
   trigger: React.ReactNode;
   entityId: string;
   spaceId?: string;
   verified?: boolean;
   onDone: (result: { id: string; name: string | null; space?: string; verified?: boolean }) => void;
 };
 
-export function SelectSpaceAsPopover({ trigger, onDone, entityId, spaceId, verified }: SelectEntityAsPopoverProps) {
+// Either both `open` and `onOpenChange` are provided (controlled) or neither is
+// (uncontrolled). Passing `open` without `onOpenChange` would leave Radix unable
+// to dismiss the popover.
+type SelectEntityAsPopoverProps = SelectEntityAsPopoverBaseProps &
+  ({ open?: undefined; onOpenChange?: undefined } | { open: boolean; onOpenChange: (open: boolean) => void });
+
+export function SelectSpaceAsPopover({
+  trigger,
+  onDone,
+  entityId,
+  spaceId,
+  verified,
+  open,
+  onOpenChange,
+}: SelectEntityAsPopoverProps) {
   return (
-    <Popover.Root>
+    <Popover.Root open={open} onOpenChange={onOpenChange}>
       <Popover.Trigger asChild>{trigger}</Popover.Trigger>
 
       <Popover.Portal>

apps/web/partials/blocks/table/collection-row-actions.tsx

@@ -46,6 +46,12 @@ export function CollectionRowActions({
   openedWithMainViewEditing = false,
 }: CollectionRowActionsProps) {
   const [isPopoverOpen, setIsPopoverOpen] = useState(false);
+  // The space-selector is a nested popover. Its content is portaled outside this
+  // popover's DOM, so opening it would normally fire `onFocusOutside` /
+  // `onInteractOutside` on the outer popover and immediately unmount the nested
+  // popover along with us. Track its open state and suppress the outer's
+  // dismiss handlers while it's active.
+  const [isSpacePopoverOpen, setIsSpacePopoverOpen] = useState(false);
   const closeTimeoutRef = useRef<ReturnType<typeof setTimeout> | null>(null);
   const { storage } = useMutate();
   const { blockEntity } = useDataBlock();
@@ -68,6 +74,21 @@ export function CollectionRowActions({
     }
   };
 
+  // A relation has two IDs: `id` (its own identifier, used for update/delete) and
+  // `entityId` (the entity that represents the relation, used for navigation —
+  // same pattern as LinkableRelationChip). Look up the collection-item relation by
+  // its id and navigate to its entityId so the link lands on the relation entity
+  // page, not on a non-existent entity at /space/.../$relation.id.
+  //
+  // Gate the scan on `isPopoverOpen`: in list/gallery views `CollectionRowActions`
+  // is mounted per row (CSS-hidden until hover), so scanning every render would be
+  // O(rows × relations). Radix only mounts `Popover.Content` (where the link
+  // lives) while open, so the lookup only needs to be correct for the open row.
+  const collectionItemRelation = isPopoverOpen
+    ? blockEntity?.relations.find(r => r.id === relationId)
+    : undefined;
+  const relationEntityId = collectionItemRelation?.entityId ?? relationId;
+
   return (
     <div className="flex shrink-0 flex-nowrap items-center gap-0.5">
       {relationId && (
@@ -106,13 +127,20 @@ export function CollectionRowActions({
                 event.preventDefault();
                 event.stopPropagation();
               }}
+              onFocusOutside={event => {
+                if (isSpacePopoverOpen) event.preventDefault();
+              }}
+              onInteractOutside={event => {
+                if (isSpacePopoverOpen) event.preventDefault();
+              }}
               onMouseEnter={() => {
                 if (closeTimeoutRef.current) {
                   clearTimeout(closeTimeoutRef.current);
                   closeTimeoutRef.current = null;
                 }
               }}
               onMouseLeave={() => {
+                if (isSpacePopoverOpen) return;
                 setIsPopoverOpen(false);
               }}
             >
@@ -121,15 +149,17 @@ export function CollectionRowActions({
                   entityId={EntityId(entityId)}
                   spaceId={spaceId}
                   verified={verified}
+                  open={isSpacePopoverOpen}
+                  onOpenChange={setIsSpacePopoverOpen}
                   onDone={result => {
                     if (!relationId) return;
                     onLinkEntry(relationId, result, verified);
+                    setIsSpacePopoverOpen(false);
                   }}
                   trigger={
                     <button
                       type="button"
                       className="inline-flex items-center p-1"
-                      onMouseDown={e => e.preventDefault()}
                     >
                       <span className="inline-flex size-[12px] items-center justify-center rounded-sm border group-hover:border-grey-03 group-hover:text-grey-03 hover:border-text! hover:text-text!">
                         {space ? (
@@ -145,7 +175,7 @@ export function CollectionRowActions({
                 />
               )}
               <PrefetchLink
-                href={`/space/${currentSpaceId}/${relationId}`}
+                href={`/space/${currentSpaceId}/${relationEntityId}`}
                 className="p-1 group-hover:text-grey-03 hover:text-text!"
               >
                 <RelationSmall />