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

Updated dependencies #2

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Updated dependencies #2

wants to merge 3 commits into from

Conversation

tsekiguchi
Copy link

Here's fully working dependencies with electron 28.2.5. Also updated electron-builder and vite-electron and vite.

In renderer, I updated svelte, svelteKit, vite added the new vitePreprocess plugin for svelteKit. Had to adjust the main +layout.svelte to add a slot.

main/index.ts got prettified.

main/lib/utils.ts updated - event.isTrusted no longer existed, so I removed the method on event

renderer/svelte.config.js updated to include the new vitePreprocess plugin.

Let me know if you have any questions about anything else or notice any errors!

@dj-nuo
Copy link
Owner

dj-nuo commented Mar 4, 2024

Apparently, when you want to use, for example, your Apple Silicon Mac to build for both Mac (universal) & Windows - you will need to stick with Electron 26.x (at least for now). It's mentioned in this issue: alex8088/electron-vite#315
I also experienced this issue & so I'm sticking with 26.x for now

@tsekiguchi
Copy link
Author

Great to know! I had no idea that issue existed - I've only been building for arm64 at this moment. I've rolled back electron to the latest 26.x and checked to make sure that electron-builder was still compatible as well.

An update to electron-vite rolled out a couple days ago, so I've updated that as well.

LMK if you see any other dependency related issues, and I'll be happy to fix!

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

Successfully merging this pull request may close these issues.

None yet

2 participants