Skip to content

Commit 9aba0b5

Browse files
committed
fix: correct path construction for light theme logo
Fix the logo path to correctly insert 'lightTheme/' directory in the right position within the assets path. Before (wrong): lightTheme/./assets/branding/che-logo.svg After (correct): ./assets/branding/lightTheme/che-logo.svg The branding.logoFile already includes the './assets/branding/' prefix, so we need to replace it with './assets/branding/lightTheme/' for light mode, not prepend 'lightTheme/' to the entire path. Assisted-by: Claude Sonnet 4.5 Signed-off-by: Oleksii Orel <oorel@redhat.com>
1 parent e7695c6 commit 9aba0b5

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

.image-tag

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
pr-1452-06
1+
pr-1452-07

packages/dashboard-frontend/src/Layout/index.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -113,9 +113,10 @@ const LayoutComponent: React.FC<Props> = props => {
113113
const { history, branding, dashboardLogo } = props;
114114

115115
// Use light theme logo for light mode, default logo for dark mode
116+
// branding.logoFile is already prefixed with './assets/branding/'
116117
const logoFile = isDarkTheme
117118
? branding.logoFile
118-
: `lightTheme/${branding.logoFile}`;
119+
: branding.logoFile.replace('./assets/branding/', './assets/branding/lightTheme/');
119120

120121
const logoSrc = buildLogoSrc(dashboardLogo, logoFile);
121122

0 commit comments

Comments
 (0)