From ecc6f3efc471e65c71036bc230330a9262d4f296 Mon Sep 17 00:00:00 2001 From: riyadhafraa <49757212-riyadhafraa@users.noreply.replit.com> 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. Replit-Commit-Author: Agent Replit-Commit-Session-Id: 77cfe984-2c65-4152-bb7a-0df28274fe66 Replit-Commit-Checkpoint-Type: full_checkpoint Replit-Commit-Event-Id: 70a0c645-f927-4c80-a04a-32ae50f404f9 Replit-Commit-Screenshot-Url: https://storage.googleapis.com/screenshot-production-us-central1/c3c252e4-c83d-40ca-9fff-99a3ea60701e/77cfe984-2c65-4152-bb7a-0df28274fe66/591TuZw Replit-Helium-Checkpoint-Created: true --- 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;