Skip to content

Commit 58b53db

Browse files
committed
chore: clean base css
1 parent 8f207d5 commit 58b53db

File tree

4 files changed

+17
-28
lines changed

4 files changed

+17
-28
lines changed

src/Pages/GlobalConfigurations/BuildInfra/ProfileForm.tsx

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@ import './styles.scss'
1414

1515
const ProfileForm: FunctionComponent = () => {
1616
const { breadcrumbs } = useBreadcrumb(BUILD_INFRA_BREADCRUMB)
17-
// Sending isSuperAdmin since don't even want to send API Request for that case
1817
const {
1918
isLoading,
2019
profileResponse,
@@ -26,13 +25,10 @@ const ProfileForm: FunctionComponent = () => {
2625
loadingActionRequest,
2726
handleSubmit,
2827
} = useBuildInfraForm({ name: DEFAULT_PROFILE_NAME, editProfile: true })
29-
// Would use this to disable the save button, if all keys in profileInputErrors are null, then there are no errors
30-
// Might enhance this check later for other operations
28+
// Would use this to disable the save button, if all keys in profileInputErrors are null or undefined, then there are no errors
29+
// Empty string means isRequired but no need to show error below input field
3130
const formErrorCount = useMemo(
32-
() =>
33-
Object.keys(profileInputErrors).filter(
34-
(item) => profileInputErrors[item] !== null && profileInputErrors[item] !== undefined,
35-
).length,
31+
() => Object.keys(profileInputErrors).filter((item) => !profileInputErrors[item]).length,
3632
[profileInputErrors],
3733
)
3834

src/components/globalConfigurations/GlobalConfiguration.tsx

Lines changed: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -486,15 +486,13 @@ function NavItem({ serverMode }) {
486486
<div className="flexbox flex-justify">Lock Deployment config</div>
487487
</NavLink>
488488
)}
489-
{BuildInfra && (
490-
<NavLink
491-
to={URLS.GLOBAL_CONFIG_BUILD_INFRA}
492-
key={URLS.GLOBAL_CONFIG_BUILD_INFRA}
493-
activeClassName="active-route"
494-
>
495-
<div className="flexbox flex-justify">Build Infra</div>
496-
</NavLink>
497-
)}
489+
<NavLink
490+
to={URLS.GLOBAL_CONFIG_BUILD_INFRA}
491+
key={URLS.GLOBAL_CONFIG_BUILD_INFRA}
492+
activeClassName="active-route"
493+
>
494+
<div className="flexbox flex-justify">Build Infra</div>
495+
</NavLink>
498496
</>
499497
)}
500498
</div>
@@ -612,7 +610,7 @@ function Body({ getHostURLConfig, checkList, serverMode, handleChecklistUpdate,
612610
<Route key={URLS.GLOBAL_CONFIG_EXTERNAL_LINKS} path={URLS.GLOBAL_CONFIG_EXTERNAL_LINKS}>
613611
<ExternalLinks />
614612
</Route>,
615-
...(serverMode !== SERVER_MODE.EA_ONLY && BuildInfra
613+
...(serverMode !== SERVER_MODE.EA_ONLY
616614
? [
617615
<Route key={URLS.GLOBAL_CONFIG_BUILD_INFRA} path={URLS.GLOBAL_CONFIG_BUILD_INFRA}>
618616
<BuildInfra isSuperAdmin={isSuperAdmin} />

src/css/base.scss

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -2108,10 +2108,6 @@ button.anchor {
21082108
border-bottom: 2px solid var(--N200);
21092109
}
21102110

2111-
.dc__border-bottom-b5-2 {
2112-
border-bottom: 2px solid var(--B500) !important;
2113-
}
2114-
21152111
.dc__border-left {
21162112
border-left: solid 1px var(--N200);
21172113
}
@@ -4126,7 +4122,7 @@ textarea::placeholder {
41264122

41274123
.bulk-selection-widget {
41284124
border: 1px solid var(--B500);
4129-
background: var(--light-blue);
4125+
background: var(--B50);
41304126
box-shadow: 0px 4px 8px 0px rgba(0, 0, 0, 0.35);
41314127
backdrop-filter: blur(5px);
41324128
}
@@ -4135,9 +4131,9 @@ textarea::placeholder {
41354131
// Tab behaviors
41364132
.dc__tab-focus {
41374133
outline: none;
4138-
}
41394134

4140-
.dc__tab-focus:focus-visible {
4141-
outline: 5px auto -webkit-focus-ring-color;
4142-
outline-offset: -2px;
4135+
&:focus-visible {
4136+
outline: 5px auto -webkit-focus-ring-color;
4137+
outline-offset: -2px;
4138+
}
41434139
}

src/css/colorPalette.scss

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
--B300: #72ace6;
1111
--B200: #abcff3;
1212
--B100: #e5f2ff;
13-
--B50: #F0F7FF;
13+
--B50: #f0f7ff;
1414
--N900: #000a14;
1515
--N800: #1d2730;
1616
--N700: #3b444c;
@@ -58,5 +58,4 @@
5858
--tippy-card-black-bg: #2d3452;
5959
--terminal-bg: #0b0f22;
6060
--inprogress-orange: #ff7e5b;
61-
--light-blue: #f0f7ff;
6261
}

0 commit comments

Comments
 (0)