fix: add useEffect cleanup function

coder · jaaydenh · Nov 6, 2024 · Oct 2, 2024 · Oct 15, 2024 · Oct 15, 2024
commit 25cbb8730c6d8f6bd007b6a1b6d42cce8cf99b62
@@ -60,12 +60,16 @@ export const ThemeProvider: FC<PropsWithChildren> = ({ children }) => {
 
 	useEffect(() => {
 		const root = document.documentElement;
-		root.classList.remove("light", "dark");
+
 		if (themePreference === "auto") {
 			root.classList.add(preferredColorScheme);
 		} else {
 			root.classList.add(themePreference);
 		}
+
+		return () => {
+			root.classList.remove("light", "dark");
+		};
 	}, [themePreference, preferredColorScheme]);
 
 	const theme =
@@ -79,6 +83,7 @@ export const ThemeProvider: FC<PropsWithChildren> = ({ children }) => {
 	);
 };
 
+// This is being added to allow Tailwind and MUI to work together.
 const cache = createCache({
 	key: "css",
 	prepend: true,

@@ -18,7 +18,7 @@ const EnterpriseVersion: FC = () => {
 					<p className="text-sm max-w-xl mt-2 text-content-secondary font-medium">
 						As an Enterprise license holder, you already benefit from Coder’s
 						features for secure, large-scale deployments. Upgrade to Coder
-						Premium for enhanced multi-tenant control and flexibility:
+						Premium for enhanced multi-tenant control and flexibility.
 					</p>
 				</div>
 				<Button asChild>