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

Svelte v4.0 Compatibility #1689

Closed
endigo9740 opened this issue Jun 22, 2023 · 9 comments · Fixed by #1690
Closed

Svelte v4.0 Compatibility #1689

endigo9740 opened this issue Jun 22, 2023 · 9 comments · Fixed by #1690
Assignees
Labels
enhancement New feature or request

Comments

@endigo9740
Copy link
Contributor

endigo9740 commented Jun 22, 2023

Svelte v4 launched today:
https://svelte.dev/blog/svelte-4

We're currently investigating support and compatibility with Skeleton. When we've identified and resolved all relevant issues we'll make an official announcement to make it clear that we provide full support for v4. Until then, it's recommend you remain on v3.

That said, if you do update to v4 and encounter issues - please report them here. We'll follow up asap. Thanks for your cooperation.

@endigo9740 endigo9740 added the enhancement New feature or request label Jun 22, 2023
@endigo9740 endigo9740 pinned this issue Jun 22, 2023
@AdrianGonz97 AdrianGonz97 mentioned this issue Jun 22, 2023
17 tasks
@ollema
Copy link

ollema commented Jun 22, 2023

maybe this is already known - when I updated to Svelte 4 I got this:

10:16:08 PM [vite-plugin-svelte] /node_modules/.pnpm/@skeletonlabs+skeleton@1.8.0/node_modules/@skeletonlabs/skeleton/dist/components/AppRail/AppRailTile.svelte:36:1 A11y: <div> with keydown, keyup, keypress handlers must have an ARIA role
10:16:08 PM [vite-plugin-svelte] /node_modules/.pnpm/@skeletonlabs+skeleton@1.8.0/node_modules/@skeletonlabs/skeleton/dist/components/Avatar/Avatar.svelte:24:0 A11y: Non-interactive element <figure> should not be assigned mouse or keyboard event listeners.
10:16:08 PM [vite-plugin-svelte] /node_modules/.pnpm/@skeletonlabs+skeleton@1.8.0/node_modules/@skeletonlabs/skeleton/dist/components/ConicGradient/ConicGradient.svelte:67:4 A11y: Non-interactive element <li> should not be assigned mouse or keyboard event listeners.
10:16:08 PM [vite-plugin-svelte] /node_modules/.pnpm/@skeletonlabs+skeleton@1.8.0/node_modules/@skeletonlabs/skeleton/dist/components/Tab/TabGroup.svelte:31:0 A11y: <div> with click, keypress, keydown, keyup handlers must have an ARIA role
10:16:08 PM [vite-plugin-svelte] /node_modules/.pnpm/@skeletonlabs+skeleton@1.8.0/node_modules/@skeletonlabs/skeleton/dist/components/TableOfContents/TableOfContents.svelte:97:4 A11y: Non-interactive element <li> should not be assigned mouse or keyboard event listeners.
10:16:08 PM [vite-plugin-svelte] /node_modules/.pnpm/@skeletonlabs+skeleton@1.8.0/node_modules/@skeletonlabs/skeleton/dist/utilities/Modal/Modal.svelte:138:2 A11y: <div> with mousedown, mouseup handlers must have an ARIA role
10:16:08 PM [vite-plugin-svelte] /node_modules/.pnpm/@skeletonlabs+skeleton@1.8.0/node_modules/@skeletonlabs/skeleton/dist/utilities/Drawer/Drawer.svelte:127:1 A11y: <div> with mousedown, keypress handlers must have an ARIA role
10:16:08 PM [vite-plugin-svelte] /node_modules/.pnpm/@skeletonlabs+skeleton@1.8.0/node_modules/@skeletonlabs/skeleton/dist/utilities/Toast/Toast.svelte:115:4 A11y: <div> with mouseenter, mouseleave handlers must have an ARIA role

@endigo9740
Copy link
Contributor Author

endigo9740 commented Jun 22, 2023

Thanks @ollema - yeah Svelte likes to adjust and change the a11y warnings in their updates. These are just warnings though, so they should not impact your ability to or deploy your application. We will look to resolve these asap though.

@gihrig
Copy link

gihrig commented Jun 24, 2023

Toast transition no longer functions.

After updating to svelte 4.0.0, toast appears and disappears instantly.
Transition works as expected with all dependencies at latest except for svelte at 3.59.1

Possibly related to:
breaking change:
"This release also makes the Svelte authoring experience more intuitive and consistent. E.g. |local is now the default for transitions to avoid animations blocking page transitions,"

See:
breaking: Make transitions local by default to prevent confusion around page navigations (sveltejs/svelte#6686)
Fix transitions so that they don't require a style-src 'unsafe-inline' Content Security Policy (CSP) (sveltejs/svelte#6662).

@endigo9740
Copy link
Contributor Author

@AdrianGonz97 as discussed on Discord, try and write up a full list of changes so we know what to test. Myself and any other volunteers that wish to join in will then run through and test as we can.

I'll try running these against the standalone projects I'm building at the moment, which should represent some real-world use cases.

@endigo9740 endigo9740 unpinned this issue Jul 4, 2023
@mmailaender
Copy link
Contributor

mmailaender commented Jul 4, 2023

I've updated to Skeleton 1.9 and getting this A11y Warning

7:01:32 PM [vite-plugin-svelte] /node_modules/.pnpm/@skeletonlabs+skeleton@1.9.0_svelte@4.0.3/node_modules/@skeletonlabs/skeleton/dist/components/Ratings/Ratings.svelte:29:3 A11y: <svelte:element> with click handler must have an ARIA role
7:01:32 PM [vite-plugin-svelte] /node_modules/.pnpm/@skeletonlabs+skeleton@1.9.0_svelte@4.0.3/node_modules/@skeletonlabs/skeleton/dist/components/Ratings/Ratings.svelte:33:3 A11y: <svelte:element> with click handler must have an ARIA role
7:01:32 PM [vite-plugin-svelte] /node_modules/.pnpm/@skeletonlabs+skeleton@1.9.0_svelte@4.0.3/node_modules/@skeletonlabs/skeleton/dist/components/Ratings/Ratings.svelte:37:3 A11y: <svelte:element> with click handler must have an ARIA role
7:01:40 PM [vite-plugin-svelte] /node_modules/.pnpm/@skeletonlabs+skeleton@1.9.0_svelte@4.0.3/node_modules/@skeletonlabs/skeleton/dist/components/Ratings/Ratings.svelte:29:3 A11y: <svelte:element> with click handler must have an ARIA role
7:01:40 PM [vite-plugin-svelte] /node_modules/.pnpm/@skeletonlabs+skeleton@1.9.0_svelte@4.0.3/node_modules/@skeletonlabs/skeleton/dist/components/Ratings/Ratings.svelte:33:3 A11y: <svelte:element> with click handler must have an ARIA role
7:01:40 PM [vite-plugin-svelte] /node_modules/.pnpm/@skeletonlabs+skeleton@1.9.0_svelte@4.0.3/node_modules/@skeletonlabs/skeleton/dist/components/Ratings/Ratings.svelte:37:3 A11y: <svelte:element> with click handler must have an ARIA role

@endigo9740
Copy link
Contributor Author

@mmailaender it looks like one of the warnings slipped through - this has been reported here:
#1708

We'll aim to address this in the following release. Thanks for the heads up!

FYI it's just a warning, so it's annoying, but it shouldn't harm your ability to use and deploy your application. You can disregard it for the time being.

@sakgoyal
Copy link

is skeleton compatible with svelte4?
Im trying to install it on my existing project and I noticed skeleton is using cjs files instead of js files.
and when I try to follow the instructions in the guide, I get errors in the tailwind config file.

@Sarenor
Copy link
Contributor

Sarenor commented Jul 23, 2023

Skeleton should be compatible with Svelte 4.
Do you want to open up an issue or discussion with your problems?
You can also try our support-discord: https://discord.com/channels/1003691521280856084/1048010639219642419
(support's usually a bit slow on weekends as well, just fyi)

@endigo9740
Copy link
Contributor Author

@sakgoyal the only .cjs file that I can think of would be for Tailwind. Right now we are focusing support on this, as this is the long standing default for Tailwind, even though they recently added support for other formats. But apples and oranges here.

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

Successfully merging a pull request may close this issue.

7 participants