fix: add useEffect cleanup function

jaaydenh · jaaydenh · commit d67f81345af0 · 2024-11-05T14:52:55.000Z
diff --git a/site/src/contexts/ThemeProvider.tsx b/site/src/contexts/ThemeProvider.tsx
@@ -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,
diff --git a/site/src/pages/DeploymentSettingsPage/PremiumPage/PremiumPageView.tsx b/site/src/pages/DeploymentSettingsPage/PremiumPage/PremiumPageView.tsx
@@ -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>
