Skip to content

Commit dbb62da

Browse files
#RI-5211 - fix style and add test id
1 parent f33716c commit dbb62da

File tree

3 files changed

+9
-5
lines changed

3 files changed

+9
-5
lines changed

redisinsight/ui/src/pages/rdi/instance/components/confirm-leave-page-popup/ConfirmLeavePagePopup.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ const ConfirmLeavePagePopup = (props: Props) => {
4343
</EuiText>
4444
</section>
4545
<div className={styles.footer}>
46-
<Download />
46+
<Download dataTestid="popup-download-pipeline-btn" />
4747
<EuiButton
4848
fill
4949
color="secondary"

redisinsight/ui/src/pages/rdi/instance/components/confirm-leave-page-popup/styles.module.scss

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
justify-content: space-between;
2424
align-items: center;
2525
padding-top: 40px;
26-
padding-bottom: 1px;
26+
padding-bottom: 6px;
2727
}
2828

2929
.button {
@@ -35,7 +35,7 @@
3535
}
3636

3737
:global {
38-
.euiModal__closeIcon {
38+
.euiButtonIcon.euiModal__closeIcon {
3939
right: 35px;
4040
top: 35px;
4141
}

redisinsight/ui/src/pages/rdi/pipeline-management/components/download/Download.tsx

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,11 @@ import { IPipeline } from 'uiSrc/slices/interfaces'
1010
import { rdiPipelineSelector } from 'uiSrc/slices/rdi/pipeline'
1111
import { sendEventTelemetry, TelemetryEvent } from 'uiSrc/telemetry'
1212

13-
const Download = () => {
13+
interface Props {
14+
dataTestid?: string
15+
}
16+
17+
const Download = ({ dataTestid }: Props) => {
1418
const { loading } = useSelector(rdiPipelineSelector)
1519

1620
const { rdiInstanceId } = useParams<{ rdiInstanceId: string }>()
@@ -45,7 +49,7 @@ const Download = () => {
4549
disabled={loading}
4650
onClick={handleDownloadClick}
4751
aria-labelledby="Download pipeline button"
48-
data-testid="download-pipeline-btn"
52+
data-testid={dataTestid || 'download-pipeline-btn'}
4953
>
5054
Download
5155
</EuiButtonEmpty>

0 commit comments

Comments
 (0)