Skip to content

Commit 9e8c35a

Browse files
authored
fix(ui5-avatar): remove redundant placeholder variables from horizon themes (#11765)
Remove `--ui5-avatar-placeholder` and `--ui5-avatar-placeholder-color` variables from horizon theme variants as they duplicate the base theme values. The base Avatar-parameters.css already defines these variables correctly using `--sapContent_ImagePlaceholderBackground` and `--sapContent_ImagePlaceholderForegroundColor` as per design specification. Fixes: #11750
1 parent 02dfbcc commit 9e8c35a

File tree

4 files changed

+0
-8
lines changed

4 files changed

+0
-8
lines changed

packages/main/src/themes/sap_horizon/Avatar-parameters.css

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@
2121
--ui5-avatar-accent8: var(--sapAvatar_8_Background);
2222
--ui5-avatar-accent9: var(--sapAvatar_9_Background);
2323
--ui5-avatar-accent10: var(--sapAvatar_10_Background);
24-
--ui5-avatar-placeholder: var(--sapContent_ImagePlaceholderBackground);
2524

2625
--ui5-avatar-accent1-color: var(--sapAvatar_1_TextColor);
2726
--ui5-avatar-accent2-color: var(--sapAvatar_2_TextColor);
@@ -33,7 +32,6 @@
3332
--ui5-avatar-accent8-color: var(--sapAvatar_8_TextColor);
3433
--ui5-avatar-accent9-color: var(--sapAvatar_9_TextColor);
3534
--ui5-avatar-accent10-color: var(--sapAvatar_10_TextColor);
36-
--ui5-avatar-placeholder-color: var(--sapAvatar_Lite_Background);
3735

3836
--ui5-avatar-accent1-border-color: var(--sapAvatar_1_BorderColor);
3937
--ui5-avatar-accent2-border-color: var(--sapAvatar_2_BorderColor);

packages/main/src/themes/sap_horizon_dark/Avatar-parameters.css

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@
2121
--ui5-avatar-accent8: var(--sapAvatar_8_Background);
2222
--ui5-avatar-accent9: var(--sapAvatar_9_Background);
2323
--ui5-avatar-accent10: var(--sapAvatar_10_Background);
24-
--ui5-avatar-placeholder: var(--sapContent_ImagePlaceholderBackground);
2524

2625
--ui5-avatar-accent1-color: var(--sapAvatar_1_TextColor);
2726
--ui5-avatar-accent2-color: var(--sapAvatar_2_TextColor);
@@ -33,7 +32,6 @@
3332
--ui5-avatar-accent8-color: var(--sapAvatar_8_TextColor);
3433
--ui5-avatar-accent9-color: var(--sapAvatar_9_TextColor);
3534
--ui5-avatar-accent10-color: var(--sapAvatar_10_TextColor);
36-
--ui5-avatar-placeholder-color: var(--sapContent_ImagePlaceholderForegroundColor);
3735

3836
--ui5-avatar-accent1-border-color: var(--sapAvatar_1_BorderColor);
3937
--ui5-avatar-accent2-border-color: var(--sapAvatar_2_BorderColor);

packages/main/src/themes/sap_horizon_hcb/Avatar-parameters.css

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@
2222
--ui5-avatar-accent8: var(--sapAvatar_8_Background);
2323
--ui5-avatar-accent9: var(--sapAvatar_9_Background);
2424
--ui5-avatar-accent10: var(--sapAvatar_10_Background);
25-
--ui5-avatar-placeholder: var(--sapContent_ImagePlaceholderBackground);
2625

2726
--ui5-avatar-accent1-color: var(--sapAvatar_1_TextColor);
2827
--ui5-avatar-accent2-color: var(--sapAvatar_2_TextColor);
@@ -34,7 +33,6 @@
3433
--ui5-avatar-accent8-color: var(--sapAvatar_8_TextColor);
3534
--ui5-avatar-accent9-color: var(--sapAvatar_9_TextColor);
3635
--ui5-avatar-accent10-color: var(--sapAvatar_10_TextColor);
37-
--ui5-avatar-placeholder-color: var(--sapAvatar_Lite_Background);
3836

3937
--ui5-avatar-accent1-border-color: var(--sapAvatar_1_BorderColor);
4038
--ui5-avatar-accent2-border-color: var(--sapAvatar_2_BorderColor);

packages/main/src/themes/sap_horizon_hcw/Avatar-parameters.css

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@
2222
--ui5-avatar-accent8: var(--sapAvatar_8_Background);
2323
--ui5-avatar-accent9: var(--sapAvatar_9_Background);
2424
--ui5-avatar-accent10: var(--sapAvatar_10_Background);
25-
--ui5-avatar-placeholder: var(--sapContent_ImagePlaceholderBackground);
2625

2726
--ui5-avatar-accent1-color: var(--sapAvatar_1_TextColor);
2827
--ui5-avatar-accent2-color: var(--sapAvatar_2_TextColor);
@@ -34,7 +33,6 @@
3433
--ui5-avatar-accent8-color: var(--sapAvatar_8_TextColor);
3534
--ui5-avatar-accent9-color: var(--sapAvatar_9_TextColor);
3635
--ui5-avatar-accent10-color: var(--sapAvatar_10_TextColor);
37-
--ui5-avatar-placeholder-color: var(--sapAvatar_Lite_Background);
3836

3937
--ui5-avatar-accent1-border-color: var(--sapAvatar_1_BorderColor);
4038
--ui5-avatar-accent2-border-color: var(--sapAvatar_2_BorderColor);

0 commit comments

Comments
 (0)