Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Desktop: Downgrade to Electron 35.7.5#13013

Merged
laurent22 merged 1 commit intolaurent22:devfrom
personalizedrefrigerator:pr/desktop/electron-35-7
Aug 22, 2025
Merged

Desktop: Downgrade to Electron 35.7.5#13013
laurent22 merged 1 commit intolaurent22:devfrom
personalizedrefrigerator:pr/desktop/electron-35-7

Conversation

@personalizedrefrigerator
Copy link
Collaborator

@personalizedrefrigerator personalizedrefrigerator commented Aug 21, 2025

Summary

This pull request reverts to the 35.x.x branch of Electron. #12951 upgraded from Electron 35.5.1 to Electron 37.3.0. The primary reason for including this upgrade in Joplin v3.4 was probably to ensure that this security patch (also in Electron 35.6.0) was applied.

This pull request downgrades to Electron 35.7.5 and thus should still include electron/electron#47369.

Notes

Possible benefits of this pull request

Drawbacks

Testing

Manual testing is currently in progress. Additional automated integration tests should run in CI (however, with a dev build of Joplin).

Plan: The following should be tested on each platform:

  • Startup: The application starts in release mode
  • Screen reader support: Verify that it's possible for the screen reader to interact with the note editor
  • External editing
  • Notifications
  • Attaching files
  • Printing
  • Drag and drop

On Windows:

  • Window appearance

The following should be tested on at least one platform:

  • Secondary windows: Check that changes from one window are reflected in the others.
  • Folder editor: Verify that it's possible to add an emoji to a folder.
  • Tag editor: It's possible to associate tags with a note using the tag dropdown.
  • Drawing tool: It's possible to insert and edit drawings.

Test results:

  • Linux (Fedora 42/GNOME) (To-do)
    • (⚠️) Create a new task, set a notification for a few minutes in the future. Check whether the notification triggers.
    • Verify that the Joplin AppImage builds and can be started.
    • Open a note in a new window (Markdown editor). Verify that editing the note updates the main window.
    • Attach a PDF file using the "attach" button. Verify that it renders and can be printed to PDF from the note viewer.
    • Drag and drop an file from file explorer into the Rich Text editor in a new note. Repeat for the Markdown editor.
    • Create a new subnotebook using the right-click menu.
    • Open a note in an external editor. Make a change and save. Verify that the note updates based on the external change.
    • Verify that it's possible to add an emoji as a folder icon through right-click > edit.
    • Verify that clicking a resource link opens it in an external application.
      • Tested with a JSON file
    • Enable the Orca screen reader. Verify that Orca can read the note editor's content.
  • Windows 11 (To-do)
  • MacOS 15 (To-do)

Note: Post-merge, I have checked that the Windows and MacOS applications can be built, installed, and started.

@personalizedrefrigerator personalizedrefrigerator changed the title Desktop: Downgrade to Electron 35.x.x Desktop: Downgrade to Electron 35.7.5 Aug 21, 2025
@laurent22 laurent22 merged commit 37422f3 into laurent22:dev Aug 22, 2025
11 checks passed
@laurent22
Copy link
Owner

Thanks for the update. I guess we can then move to Electron 37 from v3.5 so that there's plenty of time to beta test it.

@JGCarroll
Copy link

FWIW for the Linux users, the snap has had 15,000 updates for 3.4.6 in the past 48 hours; whilst testing in stable isn't ideal, through sheer numbers alone I'd argue that's substantially more testing than the beta releases would have through numbers alone.

If there's further releases in the 3.4 series then there's a chance I'll revert this commit for the snap specifically because more people have used the newer Electron than the older one IMO and the regressions aren't applicable (e.g., Windows title bar) or will happen anyway down the line (WSL2) - the next Electron version bump is likely to include Wayland by default (Electron 39) so I think there's some value in having Electron 37 decently tested for Joplin 3.5 so there's not too much at once.

Let me know if that sounds too much but I think given realistic risk modelling, it's a decent compromise given the current scenario.

@personalizedrefrigerator
Copy link
Collaborator Author

If there's further releases in the 3.4 series then there's a chance I'll revert this commit for the snap specifically because more people have used the newer Electron than the older one IMO and the regressions aren't applicable

That seems reasonable.

@teleportpast
Copy link

Will rolling back cause errors for people who are already on v3.4.6? (https://github.com/microsoft/winget-pkgs/tree/master/manifests/j/Joplin/Joplin/3.4.6)

@JGCarroll
Copy link

No, data formats are heavily independent of Electron.

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.

4 participants