diff --git a/redisinsight/ui/src/pages/database-analysis/DatabaseAnalysisPage.tsx b/redisinsight/ui/src/pages/database-analysis/DatabaseAnalysisPage.tsx index f10f5bafb2..b20e788efe 100644 --- a/redisinsight/ui/src/pages/database-analysis/DatabaseAnalysisPage.tsx +++ b/redisinsight/ui/src/pages/database-analysis/DatabaseAnalysisPage.tsx @@ -1,6 +1,7 @@ import React, { useEffect, useState } from 'react' import { useSelector, useDispatch } from 'react-redux' import { useParams } from 'react-router-dom' +import styled from 'styled-components' import { dbAnalysisSelector, @@ -27,6 +28,15 @@ import Header from './components/header' import DatabaseAnalysisTabs from './components/data-nav-tabs' import styles from './styles.module.scss' +// Styled component for the main container with theme border +const MainContainer = styled.div>` + border: 1px solid ${({ theme }) => theme.semantic.color.border.neutral500}; + border-radius: ${({ theme }) => theme.components.semanticContainer.sizes.M.borderRadius}; + padding: ${({ theme }) => theme.components.semanticContainer.sizes.M.padding}; + height: 100%; + overflow: auto; +` + const DatabaseAnalysisPage = () => { const { viewTab } = useSelector(analyticsSettingsSelector) const { loading: analysisLoading, data } = useSelector(dbAnalysisSelector) @@ -91,7 +101,7 @@ const DatabaseAnalysisPage = () => { } return ( -
+
{ reports={reports} data={data} /> -
+ ) } diff --git a/redisinsight/ui/src/pages/pub-sub/PubSubPage.tsx b/redisinsight/ui/src/pages/pub-sub/PubSubPage.tsx index 15853e464f..b7100c8826 100644 --- a/redisinsight/ui/src/pages/pub-sub/PubSubPage.tsx +++ b/redisinsight/ui/src/pages/pub-sub/PubSubPage.tsx @@ -1,6 +1,7 @@ import React, { useEffect, useState } from 'react' import { useDispatch, useSelector } from 'react-redux' import { useParams } from 'react-router-dom' +import styled from 'styled-components' import { connectedInstanceSelector } from 'uiSrc/slices/instances/instances' import { sendEventTelemetry, @@ -23,6 +24,21 @@ import { import styles from './styles.module.scss' +// Styled components +const MainContainer = styled.div>` + border: 1px solid ${({ theme }) => theme.semantic.color.border.neutral500}; + border-radius: 8px; +` + +const ContentPanel = styled.div` + flex-grow: 1; +` + +const FooterPanel = styled.div` + margin-top: 16px; + padding: 10px 18px 28px; +` + const PubSubPage = () => { const { name: connectedInstanceName, db } = useSelector( connectedInstanceSelector, @@ -71,8 +87,8 @@ const PubSubPage = () => { } return ( -
-
+ +
Pub/Sub @@ -82,10 +98,10 @@ const PubSubPage = () => { <div className={styles.tableWrapper}> <MessagesListWrapper /> </div> - </div> - <div className={styles.footerPanel}> + </ContentPanel> + <FooterPanel> <PublishMessage /> - </div> + </FooterPanel> <div className={styles.onboardAnchor}> <OnboardingTour options={ONBOARDING_FEATURES.FINISH} @@ -95,7 +111,7 @@ const PubSubPage = () => { <span /> </OnboardingTour> </div> - </div> + </MainContainer> ) } diff --git a/redisinsight/ui/src/pages/pub-sub/components/messages-list/EmptyMessagesList/EmptyMessagesList.tsx b/redisinsight/ui/src/pages/pub-sub/components/messages-list/EmptyMessagesList/EmptyMessagesList.tsx index 12931ae855..e164e9c1d2 100644 --- a/redisinsight/ui/src/pages/pub-sub/components/messages-list/EmptyMessagesList/EmptyMessagesList.tsx +++ b/redisinsight/ui/src/pages/pub-sub/components/messages-list/EmptyMessagesList/EmptyMessagesList.tsx @@ -6,6 +6,7 @@ import { Text } from 'uiSrc/components/base/text' import { RiIcon } from 'uiSrc/components/base/icons/RiIcon' import styles from './styles.module.scss' +import { Row } from 'uiSrc/components/base/layout/flex' export interface Props { connectionType?: ConnectionType @@ -27,10 +28,12 @@ const EmptyMessagesList = ({ Subscribe to the Channel to see all the messages published to your database </Text> - <Text className={styles.alert}> + <Row> <RiIcon type="ToastDangerIcon" className={styles.alertIcon} /> - Running in production may decrease performance and memory available - </Text> + <Text className={styles.alert}> + Running in production may decrease performance and memory available + </Text> + </Row> {connectionType === ConnectionType.Cluster && isSpublishNotSupported && ( <> <div className={styles.separator} /> diff --git a/redisinsight/ui/src/pages/pub-sub/components/subscription-panel/SubscriptionPanel.tsx b/redisinsight/ui/src/pages/pub-sub/components/subscription-panel/SubscriptionPanel.tsx index 15d87fa7a5..0dab4bfe29 100644 --- a/redisinsight/ui/src/pages/pub-sub/components/subscription-panel/SubscriptionPanel.tsx +++ b/redisinsight/ui/src/pages/pub-sub/components/subscription-panel/SubscriptionPanel.tsx @@ -102,8 +102,11 @@ const SubscriptionPanel = () => { </FlexItem> <FlexItem> <Row align="center"> + <FlexItem> + <ClickableAppendInfo /> + </FlexItem> <FlexItem className={styles.channels}> - <FormField additionalText={<ClickableAppendInfo />}> + <FormField> <TextInput value={channels} disabled={isSubscribed} diff --git a/redisinsight/ui/src/pages/pub-sub/components/subscription-panel/components/clickable-append-info/ClickableAppendInfo.tsx b/redisinsight/ui/src/pages/pub-sub/components/subscription-panel/components/clickable-append-info/ClickableAppendInfo.tsx index 0b4da42c8d..1df265c7bd 100644 --- a/redisinsight/ui/src/pages/pub-sub/components/subscription-panel/components/clickable-append-info/ClickableAppendInfo.tsx +++ b/redisinsight/ui/src/pages/pub-sub/components/subscription-panel/components/clickable-append-info/ClickableAppendInfo.tsx @@ -45,6 +45,7 @@ const ClickableAppendInfo = () => { <br /> Supported glob-style patterns are described  <Link + variant="small-inline" target="_blank" href={getUtmExternalLink(EXTERNAL_LINKS.pubSub, { medium: UTM_MEDIUMS.Main, diff --git a/redisinsight/ui/src/pages/pub-sub/components/subscription-panel/components/clickable-append-info/styles.module.scss b/redisinsight/ui/src/pages/pub-sub/components/subscription-panel/components/clickable-append-info/styles.module.scss index 0a218c46fd..71d9ebf269 100644 --- a/redisinsight/ui/src/pages/pub-sub/components/subscription-panel/components/clickable-append-info/styles.module.scss +++ b/redisinsight/ui/src/pages/pub-sub/components/subscription-panel/components/clickable-append-info/styles.module.scss @@ -1,15 +1,8 @@ .infoIcon { - display: flex !important; - align-items: center; - justify-content: center; width: 34px; - color: var(--iconsDefaultColor) !important; - background-color: var(--browserTableRowEven) !important; } .popover { max-width: 250px !important; - background-color: var(--euiTooltipBackgroundColor) !important; - color: var(--euiTooltipTitleTextColor) !important; border-radius: 4px; } diff --git a/redisinsight/ui/src/pages/pub-sub/styles.module.scss b/redisinsight/ui/src/pages/pub-sub/styles.module.scss index 7734916fcc..bafb102b64 100644 --- a/redisinsight/ui/src/pages/pub-sub/styles.module.scss +++ b/redisinsight/ui/src/pages/pub-sub/styles.module.scss @@ -4,20 +4,6 @@ display: flex; flex-direction: column; - .contentPanel, - .footerPanel { - background-color: var(--euiColorEmptyShade); - } - - .contentPanel { - flex-grow: 1; - } - - .footerPanel { - margin-top: 16px; - padding: 10px 18px 28px; - } - .header { padding: 24px 18px 12px; border-bottom: 1px solid var(--separatorColor);