Skip to content
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

Using Vite 4 Modal Failing to Open #157

Closed
codylindley opened this issue Dec 15, 2022 · 6 comments
Closed

Using Vite 4 Modal Failing to Open #157

codylindley opened this issue Dec 15, 2022 · 6 comments

Comments

@codylindley
Copy link

FYI Something is failing, when using Vite 4.

Click modal, nothing happens.

https://stackblitz.com/edit/angular-vmqvmy-lv8str?file=src%2FApp.tsx,package.json

But Vite Build, it works.

Change Vite to 3.2.5 and works again.

@NinoM4ster
Copy link

NinoM4ster commented Dec 15, 2022

Can confirm, just tried updating all packages in a project and the vite v4.0.1 + @suid/vite-plugin v0.1.0 combination is giving me the error Make sure your app is wrapped in a <Router />, even though it's definitely wrapped.

vite v3.2.5 + @suid/vite-plugin v0.1.0 works as expected. (I haven't tested vite build on either versions)

Here's some other console warnings that I haven't seen before this:

You appear to have multiple instances of Solid. This can lead to unexpected behavior.

cleanups created outside a `createRoot` or `render` will never be run

computations created outside a `createRoot` or `render` will never be disposed

@NinoM4ster
Copy link

NinoM4ster commented Dec 15, 2022

Related to/Duplicate of #156

@juanrgm
Copy link
Member

juanrgm commented Dec 17, 2022

Is it a SolidJS issue? (solidjs/solid#1426)

@juanrgm
Copy link
Member

juanrgm commented Dec 20, 2022

This was fixed in Vite v4.0.2 (https://stackblitz.com/edit/angular-vmqvmy-uqqlfj?file=package.json).

@juanrgm juanrgm closed this as completed Dec 20, 2022
@NinoM4ster
Copy link

@juanrgm just noting that pnpm still shows a warning with vite 4.0.2:

image

Is testing under way to bump the dependency version?

@IronGeek
Copy link

@juanrgm if @suid/vite-plugin does work with vite v4.0.2, please bump the peer dependency version to:

  "vite": "^3.0.0 || ^4.0.2"

right now it still consider vite 4 as unmet dependency

"peerDependencies": {
"vite": "^3.0.0"
},

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants