From 4a8f13d868f4287aa3e5f606bb473f1e8c6cbe3c Mon Sep 17 00:00:00 2001 From: Valentin Kirilov Date: Mon, 6 Oct 2025 11:26:55 +0300 Subject: [PATCH 1/6] fix(ui): remove additional background color overrides changng the styles of the popovers re RI-7598 --- .../ui/src/styles/components/_popover.scss | 17 ----------------- 1 file changed, 17 deletions(-) diff --git a/redisinsight/ui/src/styles/components/_popover.scss b/redisinsight/ui/src/styles/components/_popover.scss index 5e3df7aceb..4b4331a915 100644 --- a/redisinsight/ui/src/styles/components/_popover.scss +++ b/redisinsight/ui/src/styles/components/_popover.scss @@ -35,22 +35,5 @@ } .popoverLikeTooltip { - border: none; max-width: 267px !important; - background-color: var(--euiTooltipBackgroundColor) !important; - .euiPopover__panelArrow.euiPopover__panelArrow--bottom:after { - border-bottom-color: var(--euiTooltipBackgroundColor) !important; - } - - .euiPopover__panelArrow.euiPopover__panelArrow--right:after { - border-right-color: var(--euiTooltipBackgroundColor) !important; - } - - .euiPopover__panelArrow.euiPopover__panelArrow--left:after { - border-left-color: var(--euiTooltipBackgroundColor) !important; - } - - .euiPopover__panelArrow.euiPopover__panelArrow--top:after { - border-top-color: var(--euiTooltipBackgroundColor) !important; - } } From c5e601ef04563ebb5cfa53bb38496a483b548184 Mon Sep 17 00:00:00 2001 From: Valentin Kirilov Date: Mon, 6 Oct 2025 11:27:29 +0300 Subject: [PATCH 2/6] fix(ui): added spacing between the buttons in the tutorial popovers --- .../CodeButtonBlock/components/RunConfirmationPopover.tsx | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/redisinsight/ui/src/components/markdown/CodeButtonBlock/components/RunConfirmationPopover.tsx b/redisinsight/ui/src/components/markdown/CodeButtonBlock/components/RunConfirmationPopover.tsx index 0fae201177..7bac22935e 100644 --- a/redisinsight/ui/src/components/markdown/CodeButtonBlock/components/RunConfirmationPopover.tsx +++ b/redisinsight/ui/src/components/markdown/CodeButtonBlock/components/RunConfirmationPopover.tsx @@ -7,6 +7,7 @@ import { setDBConfigStorageField } from 'uiSrc/services' import { ConfigDBStorageItem } from 'uiSrc/constants/storage' import { FeatureFlagComponent } from 'uiSrc/components' import { Spacer } from 'uiSrc/components/base/layout/spacer' +import { Row } from 'uiSrc/components/base/layout/flex' import { PrimaryButton, SecondaryButton, @@ -67,7 +68,7 @@ const RunConfirmationPopover = ({ onApply }: Props) => { aria-label="checkbox do not show agan" />
-
+ { > Run -
+
) From ce0a6251965423c53e2b0b7c8e4f4bd74a93f48a Mon Sep 17 00:00:00 2001 From: Valentin Kirilov Date: Mon, 6 Oct 2025 11:32:47 +0300 Subject: [PATCH 3/6] refactor(ui): cleanup no longer used styles --- .../components/load-sample-data/styles.module.scss | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/redisinsight/ui/src/pages/browser/components/load-sample-data/styles.module.scss b/redisinsight/ui/src/pages/browser/components/load-sample-data/styles.module.scss index 5b932ca159..3a04794d5d 100644 --- a/redisinsight/ui/src/pages/browser/components/load-sample-data/styles.module.scss +++ b/redisinsight/ui/src/pages/browser/components/load-sample-data/styles.module.scss @@ -1,15 +1,3 @@ .popover { min-width: 380px !important; } - -.buttonWrapper { - .loadDataBtn { - height: 36px !important; - - :global(.euiButton__text) { - font-size: 14px !important; - font-weight: 400 !important; - } - } -} - From 464c02ac7dfcdedb954815da6b402b5be41c4e5f Mon Sep 17 00:00:00 2001 From: Valentin Kirilov Date: Mon, 6 Oct 2025 13:02:03 +0300 Subject: [PATCH 4/6] fix(ui): added spacing between the buttons in the JSON key override popover --- .../components/add-item/ConfirmOverwrite.tsx | 9 +++++---- .../components/rejson-details/styles.module.scss | 14 -------------- 2 files changed, 5 insertions(+), 18 deletions(-) diff --git a/redisinsight/ui/src/pages/browser/modules/key-details/components/rejson-details/components/add-item/ConfirmOverwrite.tsx b/redisinsight/ui/src/pages/browser/modules/key-details/components/rejson-details/components/add-item/ConfirmOverwrite.tsx index 4c64944ceb..ed45950d93 100644 --- a/redisinsight/ui/src/pages/browser/modules/key-details/components/rejson-details/components/add-item/ConfirmOverwrite.tsx +++ b/redisinsight/ui/src/pages/browser/modules/key-details/components/rejson-details/components/add-item/ConfirmOverwrite.tsx @@ -7,7 +7,8 @@ import { } from 'uiSrc/components/base/forms/buttons' import { Text } from 'uiSrc/components/base/text' import { RiPopover } from 'uiSrc/components/base' -import styles from '../../styles.module.scss' +import { Row } from 'uiSrc/components/base/layout/flex' +import { Spacer } from 'uiSrc/components/base/layout' interface ConfirmOverwriteProps { isOpen: boolean @@ -37,8 +38,8 @@ const ConfirmOverwrite = ({ You already have the same JSON key. If you proceed, a value of the existing JSON key will be overwritten. - -
+ + Overwrite -
+ ) diff --git a/redisinsight/ui/src/pages/browser/modules/key-details/components/rejson-details/styles.module.scss b/redisinsight/ui/src/pages/browser/modules/key-details/components/rejson-details/styles.module.scss index 38dfe3d662..8d44d12473 100644 --- a/redisinsight/ui/src/pages/browser/modules/key-details/components/rejson-details/styles.module.scss +++ b/redisinsight/ui/src/pages/browser/modules/key-details/components/rejson-details/styles.module.scss @@ -269,20 +269,6 @@ max-width: none; } -.confirmDialogActions { - display: flex; - justify-content: flex-end; - margin-top: 16px; - - button { - margin-left: 8px; - - &:first-of-type { - margin-left: 0; - } - } -} - .actions { margin-top: 1em; display: flex; From faa04e5bc627465dcd80476191493c0553025ccd Mon Sep 17 00:00:00 2001 From: Valentin Kirilov Date: Mon, 6 Oct 2025 13:27:52 +0300 Subject: [PATCH 5/6] fix(ui): fix position of info button when removing multiple fields from list key --- .../RemoveListElements.tsx | 42 +++++++++++-------- 1 file changed, 24 insertions(+), 18 deletions(-) diff --git a/redisinsight/ui/src/pages/browser/modules/key-details/components/list-details/remove-list-elements/RemoveListElements.tsx b/redisinsight/ui/src/pages/browser/modules/key-details/components/list-details/remove-list-elements/RemoveListElements.tsx index 581bc827a9..c8f6f555f9 100644 --- a/redisinsight/ui/src/pages/browser/modules/key-details/components/list-details/remove-list-elements/RemoveListElements.tsx +++ b/redisinsight/ui/src/pages/browser/modules/key-details/components/list-details/remove-list-elements/RemoveListElements.tsx @@ -261,24 +261,30 @@ const RemoveListElements = (props: Props) => { /> - - } - > - - - + + + + + + + + {!canRemoveMultiple ? ( + {InfoBoxPopover()} + ) : ( + <> + )} + From 8fa8548c3b4c703d394bc13114f6c2a5a00c9b60 Mon Sep 17 00:00:00 2001 From: Valentin Kirilov Date: Mon, 6 Oct 2025 14:30:51 +0300 Subject: [PATCH 6/6] fix(ui): update popover notifications spacing --- .../notifications-center/Notification/Notification.tsx | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/redisinsight/ui/src/components/navigation-menu/components/notifications-center/Notification/Notification.tsx b/redisinsight/ui/src/components/navigation-menu/components/notifications-center/Notification/Notification.tsx index 06af7291da..86a07f1e8d 100644 --- a/redisinsight/ui/src/components/navigation-menu/components/notifications-center/Notification/Notification.tsx +++ b/redisinsight/ui/src/components/navigation-menu/components/notifications-center/Notification/Notification.tsx @@ -10,6 +10,7 @@ import { truncateText } from 'uiSrc/utils' import { FlexItem, Row } from 'uiSrc/components/base/layout/flex' import { TitleSize, Title } from 'uiSrc/components/base/text/Title' import { Text } from 'uiSrc/components/base/text' +import { Spacer } from 'uiSrc/components/base/layout' import { RiBadge } from 'uiSrc/components/base/display/badge/RiBadge' import styles from '../styles.module.scss' @@ -31,10 +32,9 @@ const Notification = (props: Props) => { > {notification.title} - + @@ -43,7 +43,7 @@ const Notification = (props: Props) => { - + {format(notification.timestamp * 1000, NOTIFICATION_DATE_FORMAT)}