Skip to content

Conversation

@PikachuEXE
Copy link
Member

Pull Request Type
Please select what type of pull request this is:

  • Feature Implementation

Related issue
Might or might not fix issue in #2113

Closes #2358

Description
16.x is not supported anymore
From https://www.electronjs.org/blog/electron-17-0
image

Breaking change in 17.x =

Breaking change in 18.x =

Should be safe to upgrade if true

Screenshots (if appropriate)
N/A

Testing (for code that is not small enough to be easily understandable)
Open app, open new window via

  • Keyboard shortcut
  • New window button
  • Middle click on links

Desktop (please complete the following information):

  • OS: MacOS
  • OS Version: 12.4
  • FreeTube version: 6344228

Additional context
Add any other context about the problem here.

@PrestonN PrestonN enabled auto-merge (squash) June 25, 2022 08:18
@PikachuEXE PikachuEXE force-pushed the update/electron/18.x branch from eb2eb7f to 51c9c5c Compare June 25, 2022 14:29
@absidue absidue added the PR: waiting for review For PRs that are complete, tested, and ready for review label Jun 25, 2022
@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc added the PR: dependencies Pull requests that update a dependency file label Jun 30, 2022
@wildwestrom
Copy link

This should also fix #2190.

@ChunkyProgrammer
Copy link
Member

17 will be reaching end of life soon so if there's no issues with this one then I'd prefer this PR gets merged

@PikachuEXE
Copy link
Member Author

If most people prefer 19 I can make a PR for 19 lol

@PeterDaveHello
Copy link
Contributor

Will also improve the security issue in the dependency, like these two CVEs:

@absidue
Copy link
Member

absidue commented Jul 17, 2022

This will require a bunch of testing on linux as electron 18 caused the issues in #2163.

@wildwestrom
Copy link

wildwestrom commented Jul 17, 2022

I'm currently testing this on my GNU/Linux system. Will let you know if I get any crashes.
Also running with the wayland ozone platform under wlroots.

@absidue
Copy link
Member

absidue commented Jul 20, 2022

If anyone wants to try this, you can download a build of it here: https://github.com/absidue/FreeTube/actions/runs/2703009618

Tried this in a Ubuntu 22.04 VMware VM and noticed some weird rendering issues.

The toggle loop button disappears when the video is full screen:
toggle_loop_button_disappearing

The rendering of the devtools is completely broken too:
broken_devtools_rendering

Unfortunately this seems to be an upstream issue with Electron as Visual Studio Code has the same issue: microsoft/vscode#154705

@wildwestrom
Copy link

When I middle click to open a video in a new window, the application crashes.

[53332:0720/074634.363325:ERROR:wayland_event_watcher.cc(25)] libwayland: xdg_surface@38: error 3: xdg_surface has never been configured

@PikachuEXE PikachuEXE force-pushed the update/electron/18.x branch from 51c9c5c to 9a2cf60 Compare August 5, 2022 11:29
auto-merge was automatically disabled August 26, 2022 07:46

Pull request was closed

@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Aug 26, 2022
@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc added the electron being electron Electron related bugs and trickeries label Aug 26, 2022
@PikachuEXE PikachuEXE deleted the update/electron/18.x branch September 1, 2022 01:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

electron being electron Electron related bugs and trickeries PR: dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants