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

Upgrade Electron to v30 #7000

Merged
merged 3 commits into from
Jun 13, 2024
Merged

Conversation

mook-as
Copy link
Contributor

@mook-as mook-as commented Jun 5, 2024

This upgrades Electron; fixes #6744

There were surprisingly few issues this round. I recommend reviewing commit-by-commit.

Filed #6999 to re-enable Windows ASAR integrity (because it's not supported on current electron-builder).

This is not expected to build; it's just a checkpoint.

Signed-off-by: Mark Yen <[email protected]>
`BrowserView` is now fully deprecated; replace its functionality using
`WebContentsView` instead.

- Change the layout so that we determine the extensions view size by using
  a `<div>` of the desired size (hidden behind other content).
- Since `WebContentsView` has no `setAutoResize`, manually resize the
  extensions view when the main content view size has changed, as well as
  on zoom change.
- Rewrite zoom implementation so it adjusts the zoom of all child views
  directly instead of using keyboard events.
- Use custom menu even on development builds (so that our new zoom code can
  be correctly triggered).

Signed-off-by: Mark Yen <[email protected]>
Electron-builder doesn't yet support asar integrity on Windows; disable it
so that the packaged version can start up.  This is new on Windows in
Electron 30 (previously it was only possible on macOS).

Signed-off-by: Mark Yen <[email protected]>
@gunamata gunamata requested a review from Nino-K June 6, 2024 16:52
Copy link
Member

@Nino-K Nino-K left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested it and I did not see any visual changes.

@Nino-K Nino-K merged commit e986f9a into rancher-sandbox:main Jun 13, 2024
15 checks passed
@mook-as mook-as deleted the electron-upgrade branch July 2, 2024 18:30
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.

Bump Electron to v30
2 participants