Skip to content

Commit ceaaf17

Browse files
authored
ThemeProvider: Fix setColorMode (#1936)
* Pull out resolvedColorMode to a different effect * Create pretty-cougars-walk.md
1 parent 4dc15c3 commit ceaaf17

File tree

2 files changed

+10
-1
lines changed

2 files changed

+10
-1
lines changed

.changeset/pretty-cougars-walk.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"@primer/react": patch
3+
---
4+
5+
ThemeProvider: Fix `setColorMode`. Broken in `34.6.0`

src/ThemeProvider.tsx

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,11 @@ export const ThemeProvider: React.FC<ThemeProviderProps> = ({children, ...props}
9999
// Update state if props change
100100
React.useEffect(() => {
101101
setColorMode(props.colorMode ?? fallbackColorMode ?? defaultColorMode)
102-
}, [props.colorMode, resolvedColorMode, fallbackColorMode])
102+
}, [props.colorMode, fallbackColorMode])
103+
104+
React.useEffect(() => {
105+
setColorMode(resolvedColorMode)
106+
}, [resolvedColorMode])
103107

104108
React.useEffect(() => {
105109
setDayScheme(props.dayScheme ?? fallbackDayScheme ?? defaultDayScheme)

0 commit comments

Comments
 (0)