Skip to content

Commit d244136

Browse files
jungcome7claude
andcommitted
refactor: clean up DSThemeProvider comments
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 04f2436 commit d244136

File tree

2 files changed

+2
-5
lines changed

2 files changed

+2
-5
lines changed

apps/extension/src/theme/index.tsx

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -104,9 +104,8 @@ export const AppThemeProvider: FunctionComponent<PropsWithChildren> = ({
104104
}}
105105
>
106106
<DSThemeProvider defaultTheme={displayTheme}>
107-
<ThemeProvider theme={{ mode: displayTheme }}>
108-
{children}
109-
</ThemeProvider>
107+
{/* TODO: Remove after DS migration is complete */}
108+
<ThemeProvider theme={{ mode: displayTheme }}>{children}</ThemeProvider>
110109
</DSThemeProvider>
111110
</AppThemeContext.Provider>
112111
);

packages/design-system/src/theme/ds-theme-provider.tsx

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@ export const DSThemeProvider: React.FC<DSThemeProviderProps> = ({
2323
defaultTheme = "dark",
2424
externalMode = false,
2525
}) => {
26-
// Inject static stylesheet once (contains both dark and light rules)
2726
useLayoutEffect(() => {
2827
if (externalMode) return;
2928
const styleId = "ds-theme-vars";
@@ -35,7 +34,6 @@ export const DSThemeProvider: React.FC<DSThemeProviderProps> = ({
3534
}
3635
}, [externalMode]);
3736

38-
// Switch active theme via data attribute
3937
useLayoutEffect(() => {
4038
document.documentElement.setAttribute("data-ds-theme", defaultTheme);
4139
}, [defaultTheme]);

0 commit comments

Comments
 (0)