Dependence on window about svelte-themer HOT 8 CLOSED

I went ahead and updated the event listener. For the context key it appears since it's used in the reactive statement to update the context it's showing that warning. That key does not change therefore I think it's safe to ignore that warning for now. I'll get this merged!

from svelte-themer.

Hi @RamiAwar thank you for the kind words! Yes SSR support would be highly beneficial I'll take a look at removing the dependencies

from svelte-themer.

@RamiAwar I've pushed an update to svelte-themer@next which should resolve the issue. Once I am able to confirm I'll publish to 0.4.4

from svelte-themer.

merged and available under 0.4.5! let me know if you run into issues! closing for now

from svelte-themer.

That was fast! I'll test it now!

from svelte-themer.

It's all good on my end! Thanks!

from svelte-themer.

Also in case you haven't seen this already, this is the only warning I see in your code.

"CONTEXT_KEY" is declared in a module script and will not be reactive - svelte(module-script-reactive-declaration)

from svelte-themer.

Also, MediaQueryList.addListener seems deprecated, can be replaced by:

    darkSchemeQuery.addEventListener(
      "change",
      ({ matches }) =>
        mode === "auto" && currentMode.set(matches ? "dark" : "light")
    );

from svelte-themer.