From ea97457b031c6cfffd98f72d4964f68e6c23a331 Mon Sep 17 00:00:00 2001 From: pd-redis Date: Tue, 23 Sep 2025 13:39:00 +0300 Subject: [PATCH] disable outside click detector when ConfirmationPopover is shown Since ConfirmationPopover is "outside" the click detector "jurisdiction", clicking anywhere on it unmounts the editor and therefore, form submission does not happen --- .../ui/src/components/base/utils/OutsideClickDetector.tsx | 2 +- .../src/components/inline-item-editor/InlineItemEditor.tsx | 6 ++++-- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/redisinsight/ui/src/components/base/utils/OutsideClickDetector.tsx b/redisinsight/ui/src/components/base/utils/OutsideClickDetector.tsx index 56c23d6946..e518d98785 100644 --- a/redisinsight/ui/src/components/base/utils/OutsideClickDetector.tsx +++ b/redisinsight/ui/src/components/base/utils/OutsideClickDetector.tsx @@ -33,7 +33,7 @@ export interface OutsideClickDetectorProps { // A click event's target can be imprecise, as the value will be // the closest common ancestor of the press (mousedown, touchstart) // and release (mouseup, touchend) events (often ) if -// the the target of each event differs. +// the target of each event differs. // We need the actual event targets to make the correct decisions // about user intention. So, consider the down/start and up/end // items below as the deconstruction of a click event. diff --git a/redisinsight/ui/src/components/inline-item-editor/InlineItemEditor.tsx b/redisinsight/ui/src/components/inline-item-editor/InlineItemEditor.tsx index 65bde501ce..6501ad08dd 100644 --- a/redisinsight/ui/src/components/inline-item-editor/InlineItemEditor.tsx +++ b/redisinsight/ui/src/components/inline-item-editor/InlineItemEditor.tsx @@ -21,7 +21,6 @@ import { StyledTextInput, } from './InlineItemEditor.styles' - import styles from './styles.module.scss' type Positions = 'top' | 'bottom' | 'left' | 'right' | 'inside' @@ -216,7 +215,10 @@ const InlineItemEditor = (props: Props) => { {viewChildrenMode ? ( children ) : ( - +