search

beyonk-group / svelte-carousel

A super lightweight, super simple Carousel for Svelte 3
MIT License
213 stars 38 forks source link

TypeError: Siema is not a constructor #78

Closed Looease closed 2 years ago

Looease commented 2 years ago

Hello. I have a Svelte app and have recently updated my "rollup-plugin-svelte" to "6.1.1". Since doing this, the Carousel is no longer rendering and I am seeing the below error.

Do you have any fixes for this?

Screenshot 2022-02-11 at 11 57 42 Screenshot 2022-02-11 at 12 01 50
thobson commented 2 years ago

Same for me

mkaski commented 2 years ago

Fixed in https://github.com/beyonk-adventures/svelte-carousel/pull/79

inspiredlabs commented 2 years ago

The REPL also throws: Siema is not a constructor: https://svelte.dev/repl/3953567d530b41d087ab7eaa8e7e632a?version=3.22.3

jeremybradbury commented 2 years ago

I just got finished wiring my component wrapper up to the database / fileserver... and ofc there is some critical/breaking bug for the tool I chose.

@mkaski TYVM for fixing this! I hope we can get it approved and merged so I can get this code out of my dev environment, a custom developer build isn't going to make it to even a staging server over here.

FWIW (don't want to derail): i confirmed removing the * importing Siema gets past the first error. And I then get this one:

Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'cloneNode')
    at e.value (siema.min.js:1:4284)
    at e.value (siema.min.js:1:3696)
    at new e (siema.min.js:1:2169)
    at Carousel.svelte:32:15
    at run (index.mjs:18:12)
    at Array.map (<anonymous>)
    at index.mjs:1415:41
    at flush (index.mjs:746:17)

If I get a chance, I'll pull your fork locally and try it out.