-
-
Notifications
You must be signed in to change notification settings - Fork 5.7k
Desktop: Upgrade to Electron v37.3.0 #12951
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
Desktop: Upgrade to Electron v37.3.0 #12951
Conversation
|
Thanks for implementing this. As we are close to the release date I'd think we should move this to 3.5, or can you think of any reason to make it part of 3.4? |
|
Putting the 3.5 label for now |
Additionally, the following testing was done on MacOS 15.6: - [x] Create a new task, set a notification for a few minutes in the future. Verify that the notification triggers. - Initially, the notification wasn't triggering for me — I seem to have previously disallowed notification permissions for Joplin (in dev mode, the "allow" prompt was shown successfully). After enabling notifications in system settings, Joplin notifications were shown successfully. - [x] Open a note in a new window (Markdown editor). Verify that editing the note updates the main window. - [x] Verify that it's possible to add a tag from the secondary window. - [x] Verify that it's possible to attach a drawing from the secondary window. - [x] Attach a PDF file using the "attach" button. Verify that it renders and can be printed to PDF from the note viewer. - [x] Convert a note to PDF and open it as a preview through file > print. - [x] Create a new subnotebook using the right-click menu. - [x] Enable VoiceOver. Verify that VoiceOver reads the content of the line containing the cursor within the Markdown editor.
|
FWIW this GTK4 change is fairly significant. Prior to now, GTK3 has been stable for so long, almost every distribution has the same version of GTK3. GTK4 is another beast, where you'll find very different versions between say Ubuntu 20.04 and 24.04 or Debian 11 through 13. I don't expect most of that to matter to Joplin generally, but it does mean when diagnosing any input or window placement / etc related issues, there's now a much wider compatibility matrix on Linux than there will have been prior, even for e.g, the snap I'm there going "I wonder what this is going to look like for those users still on Ubuntu 16.04 with GTK 4.18...", only one way to find out! |
As a related note, Electron exposes a --gtk-version that should be forwarded to Electron by Joplin. |
|
This has seemingly broken the Snap on WSL2, with GTK3 and 4 Practically I don't think anyone would care, I mostly use it because I'm too lazy for proper virtual machines, but it's actually been reliable as a platform for 5 years so I thought I'd mention it incase we can draw inferences elsewhere should anything pop up. |
Summary
This pull request upgrades to Electron v37.3.0.
Breaking Electron changes
A list of related breaking changes can be found on the Electron "breaking changes" page for Electron v36 and v37. The most likely to affect Joplin are:
Testing
Automated testing: Tests in packages/app-desktop/integration-tests are run in CI (on Windows, MacOS, and Linux), but run with a development version of Joplin.
Manual testing
Plan: The following should be tested on each platform:
The following should be tested on at least one platform:
require('node-notifier').notify({ title: 'test' })also seems to do nothing.new Notification('test')from the console (which uses Electron's notification API), then the notification is shown on Windows.