-
-
Notifications
You must be signed in to change notification settings - Fork 79
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
Error occurs in the module @event-calendar/day-grid/src/Header.svelte
line 11
#179
Comments
@LeonHeidelbach This issue should be fixed in v2.3.1. Please check. |
@vkurko I can confirm that the error now no longer occurs. However, with the updated version the same issue as in theme: (theme: CalendarThemeOptions) => {
theme.button = `ec-button dark:!bg-gray-800 dark:!text-white dark:hover:!bg-gray-700
hover:cursor-pointer`;
theme.icon = 'ec-icon dark:after:!border-white';
theme.today = 'ec-today dark:!bg-gray-600 dark:!text-white';
return theme;
}, As you can see on the screenshot below, weirdly enough a couple of classes seem to still be available. Note, that even though I change the |
Oh that SvelteKit! 🙃 I'll look into this. |
@LeonHeidelbach Ok, there is v2.3.2. Lots of changes in hopes of fixing the problem. Please check. |
@vkurko Yes, this now finally does the trick. Thank you for your efforts! 👍 |
Great! 😄 |
@vkurko This error only occurs if I set a function (i.e. the one from my previous comment) to the
theme
variable and not when I set a static dictionary. So there is no way that I am passing the incorrect object. ;) Theweekdays
field does get properly set in the theme variable that is passed to the function and so does everything else (compared it to the default by console logging). From the debug output I can see that the error occurs in the module@event-calendar/day-grid/src/Header.svelte
line 11, right here:Could it be that you are not properly unsubscribing from a store on destroying a component? I am just handwaving here, since I am not familiar with your code base but the issue is definitely related to stores, funnily enough only if
theme
is set to a function though.Originally posted by @LeonHeidelbach in #155 (comment)
The text was updated successfully, but these errors were encountered: