diff --git a/frontend/src/component/feature/FeatureView2/FeatureOverview/FeatureOverviewEnvironments/FeatureOverviewEnvironment/FeatureOverviewEnvironment.styles.ts b/frontend/src/component/feature/FeatureView2/FeatureOverview/FeatureOverviewEnvironments/FeatureOverviewEnvironment/FeatureOverviewEnvironment.styles.ts index 1a3b2e2aa9..e3273dbf08 100644 --- a/frontend/src/component/feature/FeatureView2/FeatureOverview/FeatureOverviewEnvironments/FeatureOverviewEnvironment/FeatureOverviewEnvironment.styles.ts +++ b/frontend/src/component/feature/FeatureView2/FeatureOverview/FeatureOverviewEnvironments/FeatureOverviewEnvironment/FeatureOverviewEnvironment.styles.ts @@ -23,7 +23,6 @@ export const useStyles = makeStyles(theme => ({ display: 'flex', justifyContent: 'center', flexDirection: 'column', - paddingTop: '1.5rem', }, headerTitle: { @@ -133,13 +132,19 @@ export const useStyles = makeStyles(theme => ({ top: '1px', position: 'relative', }, - stratigiesIconsContainer: { + strategyIconContainer: { + minWidth: '50px', + marginRight: '5px', + display: 'flex', + justifyContent: 'center', + }, + + strategiesIconsContainer: { + transform: 'scale(0.8)', display: 'flex', alignItems: 'center', - transform: 'scale(0.8)', - top: '3px', - left: '-10px', position: 'relative', + marginLeft: '-0.75rem', [theme.breakpoints.down(560)]: { marginLeft: '0px', top: '5px', @@ -179,9 +184,7 @@ export const useStyles = makeStyles(theme => ({ padding: '0.5rem', }, }, - strategyIconContainer: { - marginRight: '5px', - }, + strategyIcon: { fill: theme.palette.grey[600], }, @@ -208,11 +211,12 @@ export const useStyles = makeStyles(theme => ({ fontWeight: 'normal', }, '&:disabled': { - margin: '0px 16px', - height: '35px' - }, + height: '35px', + background: 'transparent', + color: theme.palette.grey[600], + }, }, - separtor: { + separator: { marginLeft: '-10px', marginRight: '9px', [theme.breakpoints.down(560)]: { diff --git a/frontend/src/component/feature/FeatureView2/FeatureOverview/FeatureOverviewEnvironments/FeatureOverviewEnvironment/FeatureOverviewEnvironment.tsx b/frontend/src/component/feature/FeatureView2/FeatureOverview/FeatureOverviewEnvironments/FeatureOverviewEnvironment/FeatureOverviewEnvironment.tsx index 6163a183f4..2e76a0c060 100644 --- a/frontend/src/component/feature/FeatureView2/FeatureOverview/FeatureOverviewEnvironments/FeatureOverviewEnvironment/FeatureOverviewEnvironment.tsx +++ b/frontend/src/component/feature/FeatureView2/FeatureOverview/FeatureOverviewEnvironments/FeatureOverviewEnvironment/FeatureOverviewEnvironment.tsx @@ -116,52 +116,45 @@ const FeatureOverviewEnvironment = ({ > Add strategy - | - {getStrategyIcons()?.map( - ({ name, Icon }) => ( - -
+ + | + +
+ {getStrategyIcons()?.map( + ({ name, Icon }) => ( + - -
- - ) - )} -
- } - elseShow={ -
-

- No strategies defined on this toggle -

-
+ > + + +
+ ) + )} + {' '} + } /> @@ -187,9 +180,16 @@ const FeatureOverviewEnvironment = ({ featureEnvironment={featureEnvironment} getOverviewText={getOverviewText} /> - 0 + } + show={ + + } /> diff --git a/frontend/src/component/feature/FeatureView2/FeatureStrategies/common/GeneralStrategy/GeneralStrategy.tsx b/frontend/src/component/feature/FeatureView2/FeatureStrategies/common/GeneralStrategy/GeneralStrategy.tsx index 07dbc6fd90..9b6d8a9b1f 100644 --- a/frontend/src/component/feature/FeatureView2/FeatureStrategies/common/GeneralStrategy/GeneralStrategy.tsx +++ b/frontend/src/component/feature/FeatureView2/FeatureStrategies/common/GeneralStrategy/GeneralStrategy.tsx @@ -95,7 +95,8 @@ const GeneralStrategy = ({ ); } else if (type === 'number') { const regex = new RegExp('^\\d+$'); - const error = value.length > 0 ? !regex.test(value) : false; + const error = + value?.length > 0 ? !regex.test(value) : false; return (