From a4c591a753a75dbd45f6efc32d3fb37fba529e4b Mon Sep 17 00:00:00 2001 From: Konstantin Gogov Date: Thu, 19 Jun 2025 16:10:06 +0300 Subject: [PATCH] fix(ui5-avatar): remove redundant placeholder variables from horizon themes 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 --- packages/main/src/themes/sap_horizon/Avatar-parameters.css | 2 -- packages/main/src/themes/sap_horizon_dark/Avatar-parameters.css | 2 -- packages/main/src/themes/sap_horizon_hcb/Avatar-parameters.css | 2 -- packages/main/src/themes/sap_horizon_hcw/Avatar-parameters.css | 2 -- 4 files changed, 8 deletions(-) diff --git a/packages/main/src/themes/sap_horizon/Avatar-parameters.css b/packages/main/src/themes/sap_horizon/Avatar-parameters.css index d0991867fa1a..f6964b39091c 100644 --- a/packages/main/src/themes/sap_horizon/Avatar-parameters.css +++ b/packages/main/src/themes/sap_horizon/Avatar-parameters.css @@ -21,7 +21,6 @@ --ui5-avatar-accent8: var(--sapAvatar_8_Background); --ui5-avatar-accent9: var(--sapAvatar_9_Background); --ui5-avatar-accent10: var(--sapAvatar_10_Background); - --ui5-avatar-placeholder: var(--sapContent_ImagePlaceholderBackground); --ui5-avatar-accent1-color: var(--sapAvatar_1_TextColor); --ui5-avatar-accent2-color: var(--sapAvatar_2_TextColor); @@ -33,7 +32,6 @@ --ui5-avatar-accent8-color: var(--sapAvatar_8_TextColor); --ui5-avatar-accent9-color: var(--sapAvatar_9_TextColor); --ui5-avatar-accent10-color: var(--sapAvatar_10_TextColor); - --ui5-avatar-placeholder-color: var(--sapAvatar_Lite_Background); --ui5-avatar-accent1-border-color: var(--sapAvatar_1_BorderColor); --ui5-avatar-accent2-border-color: var(--sapAvatar_2_BorderColor); diff --git a/packages/main/src/themes/sap_horizon_dark/Avatar-parameters.css b/packages/main/src/themes/sap_horizon_dark/Avatar-parameters.css index 395b0eeb7952..f6964b39091c 100644 --- a/packages/main/src/themes/sap_horizon_dark/Avatar-parameters.css +++ b/packages/main/src/themes/sap_horizon_dark/Avatar-parameters.css @@ -21,7 +21,6 @@ --ui5-avatar-accent8: var(--sapAvatar_8_Background); --ui5-avatar-accent9: var(--sapAvatar_9_Background); --ui5-avatar-accent10: var(--sapAvatar_10_Background); - --ui5-avatar-placeholder: var(--sapContent_ImagePlaceholderBackground); --ui5-avatar-accent1-color: var(--sapAvatar_1_TextColor); --ui5-avatar-accent2-color: var(--sapAvatar_2_TextColor); @@ -33,7 +32,6 @@ --ui5-avatar-accent8-color: var(--sapAvatar_8_TextColor); --ui5-avatar-accent9-color: var(--sapAvatar_9_TextColor); --ui5-avatar-accent10-color: var(--sapAvatar_10_TextColor); - --ui5-avatar-placeholder-color: var(--sapContent_ImagePlaceholderForegroundColor); --ui5-avatar-accent1-border-color: var(--sapAvatar_1_BorderColor); --ui5-avatar-accent2-border-color: var(--sapAvatar_2_BorderColor); diff --git a/packages/main/src/themes/sap_horizon_hcb/Avatar-parameters.css b/packages/main/src/themes/sap_horizon_hcb/Avatar-parameters.css index 0881cb6a4d79..11c7d4f82ff4 100644 --- a/packages/main/src/themes/sap_horizon_hcb/Avatar-parameters.css +++ b/packages/main/src/themes/sap_horizon_hcb/Avatar-parameters.css @@ -22,7 +22,6 @@ --ui5-avatar-accent8: var(--sapAvatar_8_Background); --ui5-avatar-accent9: var(--sapAvatar_9_Background); --ui5-avatar-accent10: var(--sapAvatar_10_Background); - --ui5-avatar-placeholder: var(--sapContent_ImagePlaceholderBackground); --ui5-avatar-accent1-color: var(--sapAvatar_1_TextColor); --ui5-avatar-accent2-color: var(--sapAvatar_2_TextColor); @@ -34,7 +33,6 @@ --ui5-avatar-accent8-color: var(--sapAvatar_8_TextColor); --ui5-avatar-accent9-color: var(--sapAvatar_9_TextColor); --ui5-avatar-accent10-color: var(--sapAvatar_10_TextColor); - --ui5-avatar-placeholder-color: var(--sapAvatar_Lite_Background); --ui5-avatar-accent1-border-color: var(--sapAvatar_1_BorderColor); --ui5-avatar-accent2-border-color: var(--sapAvatar_2_BorderColor); diff --git a/packages/main/src/themes/sap_horizon_hcw/Avatar-parameters.css b/packages/main/src/themes/sap_horizon_hcw/Avatar-parameters.css index 0881cb6a4d79..11c7d4f82ff4 100644 --- a/packages/main/src/themes/sap_horizon_hcw/Avatar-parameters.css +++ b/packages/main/src/themes/sap_horizon_hcw/Avatar-parameters.css @@ -22,7 +22,6 @@ --ui5-avatar-accent8: var(--sapAvatar_8_Background); --ui5-avatar-accent9: var(--sapAvatar_9_Background); --ui5-avatar-accent10: var(--sapAvatar_10_Background); - --ui5-avatar-placeholder: var(--sapContent_ImagePlaceholderBackground); --ui5-avatar-accent1-color: var(--sapAvatar_1_TextColor); --ui5-avatar-accent2-color: var(--sapAvatar_2_TextColor); @@ -34,7 +33,6 @@ --ui5-avatar-accent8-color: var(--sapAvatar_8_TextColor); --ui5-avatar-accent9-color: var(--sapAvatar_9_TextColor); --ui5-avatar-accent10-color: var(--sapAvatar_10_TextColor); - --ui5-avatar-placeholder-color: var(--sapAvatar_Lite_Background); --ui5-avatar-accent1-border-color: var(--sapAvatar_1_BorderColor); --ui5-avatar-accent2-border-color: var(--sapAvatar_2_BorderColor);