Skip to content

Commit 5d66e8d

Browse files
committed
update styles for card component
1 parent 88f1e38 commit 5d66e8d

File tree

3 files changed

+39
-27
lines changed

3 files changed

+39
-27
lines changed

src/features/common/components/card-toolbar-buttons/card-toolbar-description-button/card-toolbar-description-button.component.tsx

Lines changed: 16 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,8 @@ import {
1111
CardToolbarButtonComponent,
1212
CardToolbarButtonComponentProps,
1313
} from "@/features/common/components/card-toolbar-button/card-toolbar-button.component";
14+
import { EyeOpenIcon } from "../../icons/eye-open/eye-open-icon";
15+
import { EyeCloseIcon } from "../../icons/eye-close/eye-close-icon";
1416

1517
interface CardToolbarDescriptionButtonComponentProps
1618
extends Omit<CardToolbarButtonComponentProps, "variant"> {
@@ -26,12 +28,12 @@ export const CardToolbarDescriptionButtonComponent: React.FC<
2628
const setDescriptionVisibility$ = useDebuggerStore((state) =>
2729
outputType === "header"
2830
? state.setDecodedHeaderDescriptionVisibility$
29-
: state.setDecodedPayloadDescriptionVisibility$,
31+
: state.setDecodedPayloadDescriptionVisibility$
3032
);
3133
const descriptionVisibility$ = useDebuggerStore((state) =>
3234
outputType === "header"
3335
? state.decodedHeaderDescriptionVisibility$
34-
: state.decodedPayloadDescriptionVisibility$,
36+
: state.decodedPayloadDescriptionVisibility$
3537
);
3638

3739
const [descVisibility, setDescVisibility] =
@@ -47,7 +49,7 @@ export const CardToolbarDescriptionButtonComponent: React.FC<
4749
ClaimDescriptionVisibilityValues.HIDDEN,
4850
{
4951
secure: true,
50-
},
52+
}
5153
);
5254

5355
setDescriptionVisibility$(ClaimDescriptionVisibilityValues.HIDDEN);
@@ -63,7 +65,7 @@ export const CardToolbarDescriptionButtonComponent: React.FC<
6365
ClaimDescriptionVisibilityValues.VISIBLE,
6466
{
6567
secure: true,
66-
},
68+
}
6769
);
6870

6971
setDescriptionVisibility$(ClaimDescriptionVisibilityValues.VISIBLE);
@@ -79,15 +81,22 @@ export const CardToolbarDescriptionButtonComponent: React.FC<
7981
<CardToolbarButtonComponent
8082
{...props}
8183
variant="standard"
84+
tooltipText={
85+
descVisibility === ClaimDescriptionVisibilityValues.VISIBLE
86+
? dictionary.hideDetailsButton.label
87+
: dictionary.showDetailsButton.label
88+
}
8289
onPress={
8390
descVisibility === ClaimDescriptionVisibilityValues.VISIBLE
8491
? hideDescription
8592
: showDescription
8693
}
8794
>
88-
{descVisibility === ClaimDescriptionVisibilityValues.VISIBLE
89-
? dictionary.hideDetailsButton.label
90-
: dictionary.showDetailsButton.label}
95+
{descVisibility === ClaimDescriptionVisibilityValues.VISIBLE ? (
96+
<EyeCloseIcon />
97+
) : (
98+
<EyeOpenIcon />
99+
)}
91100
</CardToolbarButtonComponent>
92101
);
93102
};

src/features/common/components/card/card.component.tsx

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ import { clsx } from "clsx";
44
import { getLocalizedSecondaryFont, MonoFont } from "@/libs/theme/fonts";
55
import { CardMessageComponent } from "@/features/common/components/card-message/card-message.component";
66
import { HeaderIcon } from "../icons/header/header-icon";
7+
import { CheckIcon } from "../icons/check/check-icon";
78

89
export interface CardComponentProps extends PropsWithChildren {
910
id: string;
@@ -150,7 +151,10 @@ export const CardComponent: React.FC<CardComponentProps> = (props) => {
150151
>
151152
{messages.success.map((line, index) => {
152153
return (
154+
<>
155+
<CheckIcon />
153156
<CardMessageComponent key={index}>{line}</CardMessageComponent>
157+
</>
154158
);
155159
})}
156160
</div>

src/features/common/components/card/card.module.scss

Lines changed: 19 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -2,16 +2,17 @@
22
@use "@/libs/theme/styles/mixins" as *;
33

44
.cardHeadline__title {
5-
grid-column: 1 / -1;
6-
display: flex;
7-
align-items: center;
8-
gap: 0.25rem;
9-
padding: 0.5rem 0;
10-
color: var(--color_fg_default);
11-
font-size: 0.875rem;
12-
line-height: 1.375rem;
13-
font-weight: 500;
14-
letter-spacing: 0.24px;
5+
grid-column: 1/-1;
6+
display: flex;
7+
align-items: center;
8+
gap: .25rem;
9+
font-family: var(--font-primary);
10+
padding-top: .75rem;
11+
color: var(--color_fg_default);
12+
font-size: 1rem;
13+
line-height: 1.15rem;
14+
font-weight: 700;
15+
letter-spacing: -.2px;
1516

1617
& * {
1718
font-size: inherit;
@@ -65,7 +66,6 @@
6566
border-radius: 0.5rem;
6667
border: 1px solid var(--color_border_bold);
6768
background: var(--color_bg_layer_alternate-bold);
68-
overflow: hidden;
6969

7070
&[data-type="output"] {
7171
background: var(--color_bg_layer_alternate);
@@ -206,19 +206,19 @@
206206
}
207207

208208
.card__status {
209+
flex: 1 1;
209210
display: flex;
210-
flex-direction: column;
211-
flex-shrink: 0;
212-
padding: 0.5rem 1rem;
213-
gap: 0.25rem;
214-
font-size: 0.875rem;
211+
align-items: center;
212+
justify-content: flex-start;
213+
padding: .75rem .75rem .5rem;
214+
gap: .25rem;
215+
font-size: .875rem;
215216
line-height: 1.375rem;
216217
color: var(--color_fg_default);
217-
border-top: 1px solid var(--color_border_bold);
218218
}
219219

220220
.card__success {
221-
221+
svg,
222222
span,
223223
p,
224224
code {
@@ -235,8 +235,7 @@
235235
}
236236

237237
.card__error {
238-
background-color: var(--color_bg_state_danger_subtle);
239-
238+
240239
span,
241240
p,
242241
code {

0 commit comments

Comments
 (0)