Skip to content

Commit cb3e31e

Browse files
Merge pull request #1838 from OpenSignLabs/updates-16124490027
dark-theme logo
2 parents 1cc017b + 400525d commit cb3e31e

File tree

2 files changed

+16
-17
lines changed

2 files changed

+16
-17
lines changed

apps/OpenSign/src/components/Header.jsx

Lines changed: 12 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ const Header = ({ showSidebar, setIsMenu, isConsole }) => {
2121
const image = localStorage.getItem("profileImg") || dp;
2222
const [isOpen, setIsOpen] = useState(false);
2323
const [applogo, setAppLogo] = useState("");
24+
const [isDarkTheme, setIsDarkTheme] = useState();
2425

2526
const toggleDropdown = () => {
2627
setIsOpen(!isOpen);
@@ -88,22 +89,15 @@ const Header = ({ showSidebar, setIsMenu, isConsole }) => {
8889

8990

9091
useEffect(() => {
91-
const updateLogoForTheme = () => {
92-
const isDarkMode =
92+
const updateThemeStatus = () => {
93+
const isDarkTheme =
9394
document.documentElement.getAttribute("data-theme") === "opensigndark";
94-
const logo = isDarkMode
95-
? "/static/js/assets/images/logo-dark.png" // Path to the dark mode logo
96-
: appInfo.applogo; // Use current logo for light mode
97-
if (applogo !== logo) {
98-
setAppLogo(logo);
99-
}
95+
setIsDarkTheme(isDarkTheme);
10096
};
101-
102-
// Set the logo immediately based on the current theme
103-
updateLogoForTheme();
97+
updateThemeStatus();
10498

10599
const observer = new MutationObserver(() => {
106-
updateLogoForTheme();
100+
updateThemeStatus();
107101
});
108102

109103
observer.observe(document.documentElement, {
@@ -112,7 +106,7 @@ const Header = ({ showSidebar, setIsMenu, isConsole }) => {
112106
});
113107

114108
return () => observer.disconnect();
115-
}, [applogo]);
109+
}, []);
116110

117111
return (
118112
<div>
@@ -129,9 +123,11 @@ const Header = ({ showSidebar, setIsMenu, isConsole }) => {
129123
<div className="h-[25px] md:h-[40px] w-auto overflow-hidden">
130124
{applogo && (
131125
<img
132-
className="object-contain h-full w-auto"
133-
src={applogo}
134-
alt="logo"
126+
src={
127+
isDarkTheme
128+
? "/static/js/assets/images/logo-dark.png"
129+
: applogo
130+
}
135131
/>
136132
)}
137133
</div>

apps/OpenSign/src/constant/Utils.jsx

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2414,7 +2414,10 @@ export const getAppLogo = async () => {
24142414
});
24152415
if (tenant) {
24162416
localStorage.setItem("appname", "OpenSign™");
2417-
return { logo: tenant?.logo, user: tenant?.user };
2417+
return {
2418+
logo: tenant?.logo,
2419+
user: tenant?.user
2420+
};
24182421
}
24192422
} catch (err) {
24202423
console.log("err in getlogo ", err);

0 commit comments

Comments
 (0)