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

Migrate from Tauri to Electron #32

Merged
merged 2 commits into from
Feb 10, 2025
Merged

Migrate from Tauri to Electron #32

merged 2 commits into from
Feb 10, 2025

Conversation

hojberg
Copy link
Member

@hojberg hojberg commented Feb 5, 2025

While truly a cool project, Tauri has proven to not be the right move for Unison at this time. The challenge revolve almost entirely around one of its major appeals: not bundling a browser rendering engine with the application. This means we can't consistently rely on web features across all platforms or older versions of platforms (testing on an old MacOS is increasingly difficult). Additionally the performance of the webview on Linux is a concern as we have users spread across all platforms.

This commit fully migrates the app to run on Electron, but is missingwork needed for distribution.

While truly a cool project, Tauri has proven to not be the right move
for Unison at this time. The challenge revolve almost entirely around
one of its major appeals: not bundling a browser rendering engine with
the application. This means we can't consistently rely on web features
across all platforms or older versions of platforms (testing on an old
MacOS is increasingly difficult). Additionally the performance of the
webview on Linux is a concern as we have users spread across all
platforms.

This commit fully migrates the app to run on Electron, but is missing
work needed for distribution.
@hojberg hojberg merged commit 6189be6 into main Feb 10, 2025
@hojberg hojberg deleted the electron branch February 10, 2025 16:15
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.

1 participant