From 51d723f3fb8fa6b92504a8772d44cc4b92fd007f Mon Sep 17 00:00:00 2001 From: Jackson Harper Date: Tue, 19 Apr 2022 14:58:04 -0700 Subject: [PATCH] Disable keyboard shorcuts when modals are open in the library --- .../components/templates/homeFeed/HomeFeedContainer.tsx | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/packages/web/components/templates/homeFeed/HomeFeedContainer.tsx b/packages/web/components/templates/homeFeed/HomeFeedContainer.tsx index e2ca5f184..a2099bb2e 100644 --- a/packages/web/components/templates/homeFeed/HomeFeedContainer.tsx +++ b/packages/web/components/templates/homeFeed/HomeFeedContainer.tsx @@ -236,8 +236,14 @@ export function HomeFeedContainer(props: HomeFeedContainerProps): JSX.Element { return } + // If any of the modals are open we disable handling keyboard shortcuts + if (labelsTarget || snoozeTarget || shareTarget) { + return + } + switch (action) { case 'showDetail': + const username = viewerData?.me?.profile.username if (username) { setActiveCardId(item.node.id) @@ -272,7 +278,6 @@ export function HomeFeedContainer(props: HomeFeedContainerProps): JSX.Element { setSnoozeTarget(item) break case 'set-labels': - console.log('setting labels target', item.node.id) setLabelsTarget(item) break } @@ -733,6 +738,7 @@ function HomeFeedGrid(props: HomeFeedContentProps): JSX.Element { if (props.labelsTarget) { props.labelsTarget.node.labels = value as (Label[] | undefined) } + break } }} onOpenChange={() => {