@flamingo-stack/openframe-frontend-core 0.0.538 → 0.0.539-snapshot.20260814140351

package/src/components/ui/reopen-ticket-modal.tsx

@@ -19,8 +19,7 @@ const REASON_MAX_LENGTH = 1000;
 
 export interface ReopenTicketSelection {
   statusId: string;
-  /** `null` — reopen without an assignee (the ticket stays unassigned). */
-  assigneeId: string | null;
+  assigneeId: string;
   /** Trimmed; `null` when the field was left blank. */
   reason: string | null;
 }
@@ -39,8 +38,9 @@ export interface ReopenTicketModalProps {
   assigneesLoading?: boolean;
   /** Pre-selected status; falls back to the first status option. */
   initialStatusId?: string | null;
-  /** Pre-selected assignee. Unlike Take Over, the assignee is OPTIONAL —
-   *  a reopened ticket may stay unassigned (it is flagged for triage). */
+  /** Pre-selected assignee (the design default restores the ticket's previous
+   *  assignee). No fallback — with none selected the CTA stays locked, same
+   *  as Take Over: both Status and Assigned are required to confirm. */
   initialAssigneeId?: string | null;
   /** Confirm in-flight: buttons lock and the modal refuses to close. */
   isPending?: boolean;
@@ -50,7 +50,8 @@ export interface ReopenTicketModalProps {
 /**
  * Reopen Ticket dialog (Figma openframe---tickets 8456-17581): shown when a
  * technician reopens a Resolved/Archived ticket. Same Status + Assigned pair
- * as {@link TakeOverTicketModal} plus an optional free-text reason.
+ * as {@link TakeOverTicketModal} — both REQUIRED before the CTA unlocks —
+ * plus an optional free-text reason.
  * Presentational — the consumer supplies the option lists, their ordering and
  * default selections, and performs the actual reopen in `onConfirm`.
  *
@@ -86,11 +87,11 @@ export function ReopenTicketModal({
   const selectedAssignee = assigneeOptions.find(o => o.value === assigneeId);
 
   const handleConfirm = () => {
-    if (!selectedStatusId || isPending) return;
+    if (!selectedStatusId || !assigneeId || isPending) return;
     const trimmedReason = reason.trim();
     onConfirm({
       statusId: selectedStatusId,
-      assigneeId: assigneeId ?? null,
+      assigneeId,
       reason: trimmedReason ? trimmedReason.slice(0, REASON_MAX_LENGTH) : null,
     });
   };
@@ -163,7 +164,7 @@ export function ReopenTicketModal({
           variant="accent"
           onClick={handleConfirm}
           loading={isPending}
-          disabled={!selectedStatusId}
+          disabled={!selectedStatusId || !assigneeId}
           className="flex-1"
         >
           Reopen Ticket

package/src/stories/ReopenTicketModal.stories.tsx

@@ -17,10 +17,10 @@ const meta = {
       description: {
         component:
           'Reopen Ticket dialog (Figma openframe---tickets 8456-17581): shown when a technician reopens a ' +
-          'Resolved/Archived ticket. Same Status + Assigned pair as TakeOverTicketModal plus an optional free-text ' +
-          'reason. Unlike Take Over, the assignee is optional - a reopened ticket may stay unassigned. ' +
-          'Presentational - the consumer supplies option lists (design default pre-selects Tech Required), ' +
-          'default selections, and performs the actual reopen in onConfirm.',
+          'Resolved/Archived ticket. Same Status + Assigned pair as TakeOverTicketModal - both required before ' +
+          'the CTA unlocks - plus an optional free-text reason. Presentational - the consumer supplies option ' +
+          'lists (design default pre-selects Tech Required), default selections, and performs the actual reopen ' +
+          'in onConfirm.',
       },
     },
   },
@@ -98,11 +98,11 @@ export const Default: Story = {
 };
 
 /**
- * No assignee - the reopen still confirms and the ticket stays unassigned
- * (flagged for triage). This is the deliberate difference from Take Over,
- * where the CTA locks until a technician is picked.
+ * No assignee pre-selected (e.g. the ticket was unassigned): the CTA stays
+ * locked until a technician is chosen - both fields are required, matching
+ * Take Over.
  */
-export const NoAssignee: Story = {
+export const NoDefaultAssignee: Story = {
   args: { ...Default.args, initialAssigneeId: null },
   render: args => <ControlledModal {...args} />,
 };

package/src/types/navigation.ts

@@ -126,6 +126,29 @@ export interface NavigationSidebarConfig {
    * expects so the placeholder and the loaded nav are the same height.
    */
   loadingRows?: { primary?: number; secondary?: number }
+  /**
+   * Host-owned content rendered above the primary navigation — an action that
+   * belongs in the nav's position but is not a nav entry, such as the desktop
+   * shell's "an update is ready" button.
+   *
+   * Kept out of `items` deliberately: an entry there is a row in the primary
+   * `<nav>` landmark, styled as one and announced as one, and none of that is
+   * true of an action that navigates nowhere. It also renders in every surface
+   * that takes this config — the sidebar AND the mobile burger menu — so a host
+   * passes it once and gets both.
+   *
+   * A render prop rather than a node because the two surfaces are different
+   * widths and only this component knows which: the 56px rail has room for a
+   * glyph and nothing else, while the burger menu is always full width. A host
+   * cannot derive `minimized` itself — it is the sidebar's own state, resolved
+   * from a persisted preference, the tablet breakpoint and any open overlay —
+   * and a container query would have to be authored in the HOST's Tailwind
+   * scan to emit anything.
+   *
+   * Rendered regardless of `loading`: it comes from host state that has nothing
+   * to do with whether the nav entries have resolved.
+   */
+  topSlot?: (state: { minimized: boolean }) => React.ReactNode
   minimized?: boolean
   onNavigate?: (path: string) => void
   onToggleMinimized?: () => void