From d56c390e8f4ebb761da41035153f86ef35baec85 Mon Sep 17 00:00:00 2001 From: Pankaj Patil Date: Tue, 3 Aug 2021 21:48:56 +0530 Subject: [PATCH 1/3] Fix All Material UI Warnings Signed-off-by: Pankaj Patil --- .../src/components/EmptyState/EmptyState.tsx | 2 +- packages/core-components/src/layout/Content/Content.tsx | 7 ++++++- packages/core-components/src/layout/Page/Page.stories.tsx | 2 +- packages/core-components/src/layout/Sidebar/Items.tsx | 2 +- .../core-components/src/layout/SignInPage/SignInPage.tsx | 4 ++-- .../AlertInsights/AlertInsightsSectionHeader.tsx | 2 +- plugins/fossa/src/components/FossaCard/FossaCard.tsx | 2 +- .../components/CustomResources/ArgoRollouts/Rollout.tsx | 8 ++++---- .../CustomResources/ArgoRollouts/RolloutDrawer.tsx | 2 +- .../components/CustomResources/DefaultCustomResource.tsx | 4 ++-- .../CustomResources/DefaultCustomResourceDrawer.tsx | 2 +- .../components/DeploymentsAccordions/DeploymentDrawer.tsx | 2 +- .../DeploymentsAccordions/DeploymentsAccordions.tsx | 8 ++++---- .../src/components/ErrorReporting/ErrorReporting.tsx | 2 +- .../src/components/IngressesAccordions/IngressDrawer.tsx | 2 +- .../IngressesAccordions/IngressesAccordions.tsx | 4 ++-- .../components/KubernetesContent/KubernetesContent.tsx | 8 ++++---- .../src/components/KubernetesDrawer/KubernetesDrawer.tsx | 2 +- .../src/components/ServicesAccordions/ServiceDrawer.tsx | 2 +- .../components/ServicesAccordions/ServicesAccordions.tsx | 4 ++-- plugins/lighthouse/src/components/Intro/index.tsx | 4 ++-- .../src/components/Incident/IncidentEmptyState.tsx | 2 +- .../src/components/SonarQubeCard/SonarQubeCard.tsx | 4 ++-- .../src/components/Incident/IncidentEmptyState.tsx | 2 +- .../techdocs/src/reader/components/TechDocsBuildLogs.tsx | 2 +- 25 files changed, 45 insertions(+), 40 deletions(-) diff --git a/packages/core-components/src/components/EmptyState/EmptyState.tsx b/packages/core-components/src/components/EmptyState/EmptyState.tsx index 5d55c5da93..d916ad9a61 100644 --- a/packages/core-components/src/components/EmptyState/EmptyState.tsx +++ b/packages/core-components/src/components/EmptyState/EmptyState.tsx @@ -44,7 +44,7 @@ export const EmptyState = ({ title, description, missing, action }: Props) => { ({ minWidth: 0, paddingTop: theme.spacing(3), paddingBottom: theme.spacing(3), - ...theme.mixins.gutters({}), + paddingLeft: theme.spacing(2), + paddingRight: theme.spacing(2), + [theme.breakpoints.up('sm')]: { + paddingLeft: theme.spacing(3), + paddingRight: theme.spacing(3), + }, }, stretch: { display: 'flex', diff --git a/packages/core-components/src/layout/Page/Page.stories.tsx b/packages/core-components/src/layout/Page/Page.stories.tsx index 0ea4303f2f..3ad9ecfe72 100644 --- a/packages/core-components/src/layout/Page/Page.stories.tsx +++ b/packages/core-components/src/layout/Page/Page.stories.tsx @@ -117,7 +117,7 @@ const DataGrid = () => ( xs={12} container spacing={2} - justify="space-between" + justifyContent="space-between" direction="row" > diff --git a/packages/core-components/src/layout/Sidebar/Items.tsx b/packages/core-components/src/layout/Sidebar/Items.tsx index d675b73a2c..b930c455b5 100644 --- a/packages/core-components/src/layout/Sidebar/Items.tsx +++ b/packages/core-components/src/layout/Sidebar/Items.tsx @@ -167,7 +167,7 @@ export const SidebarItem = forwardRef((props, ref) => { diff --git a/packages/core-components/src/layout/SignInPage/SignInPage.tsx b/packages/core-components/src/layout/SignInPage/SignInPage.tsx index 1ccce80e59..aaafae5dad 100644 --- a/packages/core-components/src/layout/SignInPage/SignInPage.tsx +++ b/packages/core-components/src/layout/SignInPage/SignInPage.tsx @@ -70,7 +70,7 @@ export const MultiSignInPage = ({ {title && } - + diff --git a/plugins/fossa/src/components/FossaCard/FossaCard.tsx b/plugins/fossa/src/components/FossaCard/FossaCard.tsx index a6cedfb931..58ab3163fd 100644 --- a/plugins/fossa/src/components/FossaCard/FossaCard.tsx +++ b/plugins/fossa/src/components/FossaCard/FossaCard.tsx @@ -157,7 +157,7 @@ export const FossaCard = ({ variant }: { variant?: InfoCardVariants }) => { item container direction="column" - justify="space-between" + justifyContent="space-between" alignItems="center" style={{ height: '100%' }} spacing={0} diff --git a/plugins/kubernetes/src/components/CustomResources/ArgoRollouts/Rollout.tsx b/plugins/kubernetes/src/components/CustomResources/ArgoRollouts/Rollout.tsx index ae98e4e668..46af178989 100644 --- a/plugins/kubernetes/src/components/CustomResources/ArgoRollouts/Rollout.tsx +++ b/plugins/kubernetes/src/components/CustomResources/ArgoRollouts/Rollout.tsx @@ -97,7 +97,7 @@ const RolloutSummary = ({ const abortedMessage = findAbortedMessage(rollout); return ( - + @@ -111,7 +111,7 @@ const RolloutSummary = ({ item container direction="column" - justify="flex-start" + justifyContent="flex-start" alignItems="flex-start" spacing={0} > @@ -142,7 +142,7 @@ const RolloutSummary = ({ container xs={3} direction="column" - justify="flex-start" + justifyContent="flex-start" alignItems="flex-start" > @@ -250,7 +250,7 @@ export const RolloutAccordions = ({ {rollouts.map((rollout, i) => ( diff --git a/plugins/kubernetes/src/components/CustomResources/ArgoRollouts/RolloutDrawer.tsx b/plugins/kubernetes/src/components/CustomResources/ArgoRollouts/RolloutDrawer.tsx index 2e116c0881..d719f707d1 100644 --- a/plugins/kubernetes/src/components/CustomResources/ArgoRollouts/RolloutDrawer.tsx +++ b/plugins/kubernetes/src/components/CustomResources/ArgoRollouts/RolloutDrawer.tsx @@ -35,7 +35,7 @@ export const RolloutDrawer = ({ diff --git a/plugins/kubernetes/src/components/CustomResources/DefaultCustomResource.tsx b/plugins/kubernetes/src/components/CustomResources/DefaultCustomResource.tsx index f0bd92c5bf..64c09698ac 100644 --- a/plugins/kubernetes/src/components/CustomResources/DefaultCustomResource.tsx +++ b/plugins/kubernetes/src/components/CustomResources/DefaultCustomResource.tsx @@ -51,7 +51,7 @@ const DefaultCustomResourceSummary = ({ customResourceName, }: DefaultCustomResourceSummaryProps) => { return ( - + {customResources.map((cr, i) => ( diff --git a/plugins/kubernetes/src/components/CustomResources/DefaultCustomResourceDrawer.tsx b/plugins/kubernetes/src/components/CustomResources/DefaultCustomResourceDrawer.tsx index 906e9fd616..ba6169cdf3 100644 --- a/plugins/kubernetes/src/components/CustomResources/DefaultCustomResourceDrawer.tsx +++ b/plugins/kubernetes/src/components/CustomResources/DefaultCustomResourceDrawer.tsx @@ -42,7 +42,7 @@ export const DefaultCustomResourceDrawer = ({ diff --git a/plugins/kubernetes/src/components/DeploymentsAccordions/DeploymentDrawer.tsx b/plugins/kubernetes/src/components/DeploymentsAccordions/DeploymentDrawer.tsx index 2173f9bfe4..2d22caec7c 100644 --- a/plugins/kubernetes/src/components/DeploymentsAccordions/DeploymentDrawer.tsx +++ b/plugins/kubernetes/src/components/DeploymentsAccordions/DeploymentDrawer.tsx @@ -52,7 +52,7 @@ export const DeploymentDrawer = ({ diff --git a/plugins/kubernetes/src/components/DeploymentsAccordions/DeploymentsAccordions.tsx b/plugins/kubernetes/src/components/DeploymentsAccordions/DeploymentsAccordions.tsx index 5bbeee1c64..0fa031240b 100644 --- a/plugins/kubernetes/src/components/DeploymentsAccordions/DeploymentsAccordions.tsx +++ b/plugins/kubernetes/src/components/DeploymentsAccordions/DeploymentsAccordions.tsx @@ -68,7 +68,7 @@ const DeploymentSummary = ({ hpa, }: DeploymentSummaryProps) => { return ( - + @@ -82,7 +82,7 @@ const DeploymentSummary = ({ item container direction="column" - justify="flex-start" + justifyContent="flex-start" alignItems="flex-start" spacing={0} > @@ -113,7 +113,7 @@ const DeploymentSummary = ({ container xs={3} direction="column" - justify="flex-start" + justifyContent="flex-start" alignItems="flex-start" > @@ -169,7 +169,7 @@ export const DeploymentsAccordions = ({}: DeploymentsAccordionsProps) => { {groupedResponses.deployments.map((deployment, i) => ( diff --git a/plugins/kubernetes/src/components/ErrorReporting/ErrorReporting.tsx b/plugins/kubernetes/src/components/ErrorReporting/ErrorReporting.tsx index 29107ac210..62af1d9d14 100644 --- a/plugins/kubernetes/src/components/ErrorReporting/ErrorReporting.tsx +++ b/plugins/kubernetes/src/components/ErrorReporting/ErrorReporting.tsx @@ -90,7 +90,7 @@ export const ErrorEmptyState = () => { return ( diff --git a/plugins/kubernetes/src/components/IngressesAccordions/IngressesAccordions.tsx b/plugins/kubernetes/src/components/IngressesAccordions/IngressesAccordions.tsx index d8632488ad..3cb2ee7160 100644 --- a/plugins/kubernetes/src/components/IngressesAccordions/IngressesAccordions.tsx +++ b/plugins/kubernetes/src/components/IngressesAccordions/IngressesAccordions.tsx @@ -40,7 +40,7 @@ type IngressSummaryProps = { const IngressSummary = ({ ingress }: IngressSummaryProps) => { return ( - + @@ -84,7 +84,7 @@ export const IngressesAccordions = ({}: IngressesAccordionsProps) => { {groupedResponses.ingresses.map((ingress, i) => ( diff --git a/plugins/kubernetes/src/components/KubernetesContent/KubernetesContent.tsx b/plugins/kubernetes/src/components/KubernetesContent/KubernetesContent.tsx index 004473dd63..c0dd6d5003 100644 --- a/plugins/kubernetes/src/components/KubernetesContent/KubernetesContent.tsx +++ b/plugins/kubernetes/src/components/KubernetesContent/KubernetesContent.tsx @@ -65,7 +65,7 @@ const ClusterSummary = ({ @@ -92,7 +92,7 @@ const ClusterSummary = ({ container xs={3} direction="column" - justify="flex-start" + justifyContent="flex-start" alignItems="flex-start" > @@ -208,7 +208,7 @@ export const KubernetesContent = ({ entity }: KubernetesContentProps) => { {kubernetesObjects?.items.length <= 0 && ( ({ diff --git a/plugins/kubernetes/src/components/ServicesAccordions/ServiceDrawer.tsx b/plugins/kubernetes/src/components/ServicesAccordions/ServiceDrawer.tsx index 1d214ac276..5d17fd7213 100644 --- a/plugins/kubernetes/src/components/ServicesAccordions/ServiceDrawer.tsx +++ b/plugins/kubernetes/src/components/ServicesAccordions/ServiceDrawer.tsx @@ -38,7 +38,7 @@ export const ServiceDrawer = ({ diff --git a/plugins/kubernetes/src/components/ServicesAccordions/ServicesAccordions.tsx b/plugins/kubernetes/src/components/ServicesAccordions/ServicesAccordions.tsx index f96bdc8e0b..72daba0aa5 100644 --- a/plugins/kubernetes/src/components/ServicesAccordions/ServicesAccordions.tsx +++ b/plugins/kubernetes/src/components/ServicesAccordions/ServicesAccordions.tsx @@ -35,7 +35,7 @@ type ServiceSummaryProps = { const ServiceSummary = ({ service }: ServiceSummaryProps) => { return ( - + @@ -107,7 +107,7 @@ export const ServicesAccordions = ({}: ServicesAccordionsProps) => { {groupedResponses.services.map((service, i) => ( diff --git a/plugins/lighthouse/src/components/Intro/index.tsx b/plugins/lighthouse/src/components/Intro/index.tsx index b569429b03..509f9e468a 100644 --- a/plugins/lighthouse/src/components/Intro/index.tsx +++ b/plugins/lighthouse/src/components/Intro/index.tsx @@ -105,7 +105,7 @@ function GettingStartedCard() { divider actions={ <> - +