Skip to content
This repository was archived by the owner on Jul 30, 2025. It is now read-only.

Commit 09cefc6

Browse files
committed
fix(plugins/plugin-client-common): tweaks to "mini progress" ui in wizard
to make it align with the looks of the overall progress ui
1 parent 6c64926 commit 09cefc6

File tree

3 files changed

+38
-9
lines changed

3 files changed

+38
-9
lines changed

plugins/plugin-client-common/src/components/Content/MiniProgressStepper.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ export class MiniProgressStep extends React.PureComponent<MiniProps, ProgressSte
7676
}
7777

7878
private icon() {
79-
return statusToIcon(this.status)
79+
return this.status === 'in-progress' ? statusToIcon(this.status) : <React.Fragment />
8080
}
8181

8282
private get tooltipText() {

plugins/plugin-client-common/web/scss/components/ProgressStepper/_mixins.scss

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -70,6 +70,24 @@
7070
}
7171
}
7272

73+
@mixin Success {
74+
&.pf-m-success {
75+
@content;
76+
}
77+
}
78+
79+
@mixin Warning {
80+
&.pf-m-warning {
81+
@content;
82+
}
83+
}
84+
85+
@mixin Error {
86+
&.pf-m-error {
87+
@content;
88+
}
89+
}
90+
7391
@mixin Icon {
7492
.pf-c-progress-stepper__step-icon {
7593
@content;

plugins/plugin-client-common/web/scss/components/Wizard/MiniProgressStepper.scss

Lines changed: 19 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,8 @@
1616

1717
@import '../ProgressStepper/mixins';
1818

19-
$small: 0.5625rem;
20-
$regular: 1rem;
19+
$small: 0.5rem;
20+
$regular: 0.75rem;
2121

2222
@include MiniProgressStepper {
2323
--pf-c-progress-stepper__step-icon--Width: #{$regular};
@@ -31,22 +31,33 @@ $regular: 1rem;
3131
font-size: $regular;
3232
}
3333
@include ConnectorUI {
34-
--pf-c-progress-stepper__step-connector--before--Left: 6px;
35-
top: 8px !important; /* ugh, we conflict with the stock PatternFly rules here */
34+
--pf-c-progress-stepper__step-connector--before--Left: 50%;
35+
--pf-c-progress-stepper__step-connector--before--Top: 50%;
36+
display: none;
3637
}
3738

3839
@include ProgressStep {
3940
display: block; /* Rather than `contents`, so we can wrap a Tooltip */
41+
@include Icon {
42+
border-width: 0;
43+
border-radius: 0;
44+
width: $small;
45+
height: $small;
46+
font-size: $small;
47+
}
4048
@include NotBlank {
4149
@include Icon {
4250
border: none;
4351
}
4452
}
53+
@include Success {
54+
@include Icon {
55+
background-color: var(--pf-global--primary-color--100);
56+
}
57+
}
4558
@include Blank {
4659
@include Icon {
47-
width: $small;
48-
height: $small;
49-
font-size: $small;
60+
background-color: rgba(var(--color-blue-rgb), 0.2);
5061
}
5162
}
5263
}
@@ -55,7 +66,7 @@ $regular: 1rem;
5566
@include MiniProgressStepper {
5667
--pf-c-progress-stepper--GridTemplateRows: max-content;
5768
--pf-c-progress-stepper--GridTemplateColumns: max-content;
58-
grid-gap: 3px;
69+
grid-gap: 0px;
5970
grid-auto-columns: max-content;
6071
}
6172

0 commit comments

Comments
 (0)