From 9c3985b80754577ce80b05d42628e7b70c0e53ec Mon Sep 17 00:00:00 2001 From: Riyadh Date: Mon, 11 May 2026 14:58:22 +0000 Subject: [PATCH] Prevent quick actions popover from opening when in edit mode Update conditional logic for opening the quick actions popover to account for edit mode, ensuring it only opens when mutation is allowed and edit mode is disabled. --- artifacts/tx-os/src/pages/executive-meetings.tsx | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/artifacts/tx-os/src/pages/executive-meetings.tsx b/artifacts/tx-os/src/pages/executive-meetings.tsx index 83c5d177..77d159ea 100644 --- a/artifacts/tx-os/src/pages/executive-meetings.tsx +++ b/artifacts/tx-os/src/pages/executive-meetings.tsx @@ -3840,10 +3840,13 @@ function MeetingRow({ // (no popover/edit affordances change), but useSortable is disabled // and the is dimmed + aria-disabled with a bilingual title. dayRotatable: boolean; - // #486: raw (un-gated by editMode) mutate permission — clicking a - // row anywhere outside an interactive control opens the quick- - // actions popover. The popover itself respects the same MUTATE_ROLES - // server-side gating as the postpone-minutes endpoint. + // #486/#496: mutate permission for the row's quick-actions surface + // (centered Postpone dialog opened on row click). #486 originally + // wired this un-gated by editMode; #496 narrows it to + // `canMutate && !editMode` at the call site so the dialog cannot + // open while the schedule is in edit mode (inline cell editors take + // over the row instead). The dialog itself still respects the same + // MUTATE_ROLES server-side gating as the postpone-minutes endpoint. quickActionsCanMutate?: boolean; onQuickPostpone?: () => void; onSaveTitle: (html: string) => Promise;