-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
docs(website): correct theme color blink when playground is initialized #4870
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
✅ Deploy Preview for typescript-eslint ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ezpz - thanks for this!
@@ -34,6 +35,7 @@ export const useSandboxServices = ( | |||
): Error | SandboxServices | undefined => { | |||
const [services, setServices] = useState<Error | SandboxServices>(); | |||
const [loadedTs, setLoadedTs] = useState<string>(props.ts); | |||
const { isDarkTheme } = useColorMode(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
isDarkTheme
is deprecated—you should see a warning in the console in development (it probably is not transparent enough, considering most people don't develop with their consoles open all the time). You should use const { colorMode } = useColorMode()
instead
PR Checklist
Overview