resolvedTheme can be `system` about next-themes HOT 8 CLOSED

Sweet, thanks! I wasn't able to reproduce the issue on the first render, but I was able to reproduce the issue when doing setTheme('system') twice.

@thien-do explained it in #34

since the theme value (in state) is not changed (still "system"), the "handleMediaQuery" is not triggered

I updated the logic in this commit: 8ab93d9

Basically when setting theme to system, it wasn't applying the resolved theme – that only happened by accident! This adds the check and actually makes it faster (previously there were two calls: one to update the DOM to system, then another to update the DOM to dark/light)

from next-themes.

Yeah this is really weird.. I also see this behavior, but by reading the code it should not happen https://github.com/pacocoursey/next-themes/blob/master/index.tsx !

from next-themes.

I also see it on 0.0.13, and I can reproduce it by setTheme("system") again after the initial set

from next-themes.

I think it also happened on the latest stable version (which is what prompted me to update to beta). Maybe it has to do with the default theme being system + media query not working in SSR???

from next-themes.

Yeah the latest version doesn't change the resolving part so I think it likely is.

Actually I cannot reproduce your exact issue (resolvedTheme returns "system") but I do see some related issue I noted at #34 it's when the "system" is being used for "data-theme". In my case, resolvedTheme is still correct (either "light" or "dark") but the data-theme is set incorrectly

from next-themes.

Thank you both for reporting this! I just published v0.0.13-beta.3 which should fix both #33 and #34 – please test and let me know if there is still an issue.

from next-themes.

Updating to the latest beta seems to have fixed the issue! I'm curious, what was the source of this bug?

from next-themes.

You rock @pacocoursey 🚀 it's fixed!

from next-themes.