Skip to content

Commit 3093edc

Browse files
author
Mukul Tayal
committed
css fixes from feedback
1 parent 7c58880 commit 3093edc

File tree

6 files changed

+16
-13
lines changed

6 files changed

+16
-13
lines changed

src/components/app/details/appDetails/AppDetails.tsx

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -621,7 +621,7 @@ export const Details: React.FC<DetailsType> = ({
621621

622622
return (
623623
<>
624-
<div className="w-100 pt-16 pr-20 pb-16 pl-20 app-info-bg-gradient">
624+
<div className="w-100 pt-16 pr-20 pb-16 pl-20 app-info-bg dc__gap-16">
625625
<SourceInfo
626626
appDetails={appDetails}
627627
appStreamData={streamData}
@@ -645,7 +645,7 @@ export const Details: React.FC<DetailsType> = ({
645645
setErrorsList={setErrorsList}
646646
/>
647647
</div>
648-
{!loadingDetails && !loadingResourceTree && !appDetails?.deploymentAppDeleteRequest ? (
648+
{!loadingDetails && !loadingResourceTree && !appDetails?.deploymentAppDeleteRequest && (
649649
<>
650650
{environment && !isVirtualEnvRef.current && (
651651
<AppMetrics
@@ -664,8 +664,6 @@ export const Details: React.FC<DetailsType> = ({
664664
/>
665665
)}
666666
</>
667-
) : (
668-
<div className="mb-9" />
669667
)}
670668
{loadingResourceTree ? (
671669
<div className="bcn-0 h-100">

src/components/app/details/appDetails/SourceInfo.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,7 @@ export const SourceInfo = ({
108108
loadingCards.push(<LoadingCard key={i} />)
109109
}
110110

111-
return <div className="flex left ml-20 mb-16">{loadingCards}</div>
111+
return <div className="flex left mb-16">{loadingCards}</div>
112112
}
113113

114114
const conditionalScalePodsButton = (children) => {
@@ -126,7 +126,7 @@ export const SourceInfo = ({
126126

127127
const renderDevtronAppsEnvironmentSelector = (environment) => {
128128
return (
129-
<div className="flex left w-100 mb-16">
129+
<div className="flex left w-100">
130130
<EnvSelector
131131
environments={environments}
132132
disabled={loadingDetails || loadingResourceTree || (params.envId && !showCommitInfo)}
@@ -226,7 +226,7 @@ export const SourceInfo = ({
226226
}
227227

228228
return (
229-
<div className="flex left w-100 column source-info-container">
229+
<div className="flex left w-100 column source-info-container dc__gap-16">
230230
{renderDevtronAppsEnvironmentSelector(environment)}
231231
{loadingDetails
232232
? shimmerLoaderBlocks()

src/components/app/details/appDetails/appDetails.scss

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1910,6 +1910,11 @@ table.resource-tree {
19101910
margin-left: 8px !important;
19111911
}
19121912

1913+
.app-details-manifest-download-card {
1914+
overflow: hidden;
1915+
box-shadow: 0 0 4px 0 rgba(0, 0, 0, 0.20);
1916+
}
1917+
19131918
.app-details-info-card {
19141919
align-items: flex-start;
19151920
flex-direction: column;

src/components/app/list/list.scss

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@
8181

8282
.app-list__header {
8383
display: grid;
84-
grid-template-columns: 20px 1.5fr repeat(4, 1fr) 1.5fr 76px;
84+
grid-template-columns: 24px 1.5fr repeat(4, 1fr) 1.5fr 76px;
8585
align-items: center;
8686
grid-template-rows: 36px;
8787
grid-column-gap: 16px;
@@ -104,7 +104,7 @@
104104
.app-list__row {
105105
min-height: 48px;
106106
display: grid;
107-
grid-template-columns: 20px 1.5fr repeat(4, 1fr) 1.5fr 76px;
107+
grid-template-columns: 24px 1.5fr repeat(4, 1fr) 1.5fr 76px;
108108
align-items: center;
109109
grid-column-gap: 16px;
110110
font-size: 13px;
@@ -482,7 +482,7 @@
482482
&__table-row {
483483
display: grid;
484484
align-items: center;
485-
grid-template-columns: 20px 1.5fr repeat(4, 1fr) 1.5fr 76px;
485+
grid-template-columns: 24px 1.5fr repeat(4, 1fr) 1.5fr 76px;
486486
column-gap: 16px;
487487
grid-template-rows: 40px;
488488
}

src/components/v2/appDetails/AppDetails.component.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,7 @@ const AppDetailsComponent = ({
164164

165165
return (
166166
<div className="helm-details" data-testid="app-details-wrapper">
167-
<div className="app-info-bg-gradient">
167+
<div className="app-info-bg">
168168
<EnvironmentSelectorComponent
169169
isExternalApp={isExternalApp}
170170
_init={_init}

src/components/v2/appDetails/appDetails.scss

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,6 @@
2424
flex: 1;
2525
}
2626

27-
.app-info-bg-gradient {
28-
background: linear-gradient(to top, #ffffff 0, #ffffff 50px, var(--window-bg) 50px);
27+
.app-info-bg {
28+
background: var(--window-bg);
2929
}

0 commit comments

Comments
 (0)