Skip to content

Workaround for https://github.com/electron/electron/issues/49253#8714

Merged
FreeTubeBot merged 5 commits into
FreeTubeApp:developmentfrom
Devenor:tray_workaround
Mar 11, 2026
Merged

Workaround for https://github.com/electron/electron/issues/49253#8714
FreeTubeBot merged 5 commits into
FreeTubeApp:developmentfrom
Devenor:tray_workaround

Conversation

@Devenor

@Devenor Devenor commented Feb 26, 2026

Copy link
Copy Markdown
Contributor

Pull Request Type

  • Bugfix
  • Feature Implementation
  • Documentation
  • Other

Related issue

Workaround for #8359

Description

Workaround for Electron issue electron/electron#49253, causing Freetube to crash when showing a window minimized in the tray on Linux + Wayland.

Screenshots

Testing

  1. Minimize application
  2. Go to tray
  3. Click on Show to show the minimized window
  4. The window shows

Desktop

  • OS: Ubuntu
  • OS Version: 22.04.3 LTS
  • FreeTube version: 0.23.13-beta

Additional context

@FreeTubeBot FreeTubeBot enabled auto-merge (squash) February 26, 2026 18:37
@github-actions github-actions Bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Feb 26, 2026
@absidue

absidue commented Mar 6, 2026

Copy link
Copy Markdown
Member

Could you please invert the check and flip the if-else branches as it is a bit more readable that way.

@absidue absidue added U: Waiting for Response from Author and removed PR: waiting for review For PRs that are complete, tested, and ready for review labels Mar 6, 2026
@Devenor

Devenor commented Mar 7, 2026

Copy link
Copy Markdown
Contributor Author

Sure, but I'm suddenly having some weird problems with the linter and I can't find the cause, maybe you know what it's about? It says:

[12:56:12] devenor@localhost:~/FreeTube$ yarn lint
yarn run v1.22.22
$ run-p eslint-lint lint-style
$ stylelint "src/**/*.{css,scss}"
$ eslint --config eslint.config.mjs "src/**/*.js" "src/renderer/**/*.vue" "static/*.js" "_scripts/*.js" "_scripts/**/*.mjs"
Error [ERR_REQUIRE_CYCLE_MODULE]: Cannot require() ES Module /home/devenor/FreeTube/stylelint.config.mjs in a cycle. (from /home/devenor/FreeTube/node_modules/cosmiconfig/dist/loaders.js)

@absidue

absidue commented Mar 7, 2026

Copy link
Copy Markdown
Member

Please make sure you are using Node.js 24+.

auto-merge was automatically disabled March 7, 2026 16:34

Head branch was pushed to by a user without write access

@FreeTubeBot FreeTubeBot enabled auto-merge (squash) March 7, 2026 16:35
@Devenor

Devenor commented Mar 7, 2026

Copy link
Copy Markdown
Contributor Author

Thanks, that did the trick, the PR is updated.

@ChunkyProgrammer ChunkyProgrammer added PR: waiting for review For PRs that are complete, tested, and ready for review and removed PR: changes requested labels Mar 7, 2026

@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@Devenor idk if you can do something about it but it looks very glitchy

VirtualBoxVM_yhY87kgjlE.mp4

@Devenor

Devenor commented Mar 8, 2026

Copy link
Copy Markdown
Contributor Author

I don't think it can be fixed... all I'm doing is calling maximize, and it also happens using show.

It's very smooth on my Ubuntu though (no VM). Maybe yours is worse because of the VM and running it in dev mode ? Could you try building it and running the build instead ?

@absidue

absidue commented Mar 8, 2026

Copy link
Copy Markdown
Member

@efb4f5ff-1298-471a-8973-3d47447115dc I suspect you don't have hardware acceleration/3D acceleration enabled for your VM and have animations enabled in Ubuntu (even the menu in the top right looks dodgy in your video).

@efb4f5ff-1298-471a-8973-3d47447115dc

efb4f5ff-1298-471a-8973-3d47447115dc commented Mar 10, 2026

Copy link
Copy Markdown
Member

@efb4f5ff-1298-471a-8973-3d47447115dc I suspect you don't have hardware acceleration/3D acceleration enabled for your VM and have animations enabled in Ubuntu (even the menu in the top right looks dodgy in your video).

I suspect you don't have hardware acceleration/3D acceleration enabled

This was turned on in the clip

and have animations enabled in Ubuntu (even the menu in the top right looks dodgy in your video).

This was turned off in the clip

Could you try building it and running the build instead ?

Will try

@Devenor

Devenor commented Mar 10, 2026

Copy link
Copy Markdown
Contributor Author

@efb4f5ff-1298-471a-8973-3d47447115dc I see you have the green turtle in the lower right side of your screen, that's most certainly the issue. You should try disabling Hyper-V, in my case I can do that by typing this in a command prompt as admin, and then restarting: bcdedit /set hypervisorlaunchtype off

It may not work just by doing that, though, and other steps could be necessary.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think its safe to say that these issues are unrelated to your PR. LGTM. Thanks @Devenor !

@FreeTubeBot FreeTubeBot merged commit a9747ce into FreeTubeApp:development Mar 11, 2026
6 checks passed
@github-actions github-actions Bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Mar 11, 2026
PikachuEXE added a commit to PikachuEXE/FreeTube that referenced this pull request Mar 12, 2026
* test/yt/pr-1148:
  test
  Serialize the error as a string to pass it back to the main view
  Workaround for electron/electron#49253 (FreeTubeApp#8714)
  Translated using Weblate (Icelandic)
  Translated using Weblate (Japanese)
  Trim video titles with the local API to avoid breaking the databases (FreeTubeApp#8354)
  Translated using Weblate (Tamil)
PikachuEXE added a commit to PikachuEXE/FreeTube that referenced this pull request Mar 16, 2026
* development: (106 commits)
  Translated using Weblate (Portuguese)
  Bump electron from 40.8.0 to 41.0.2 (FreeTubeApp#8801)
  Translated using Weblate (German)
  Translated using Weblate (Lithuanian)
  Bump lefthook from 2.1.3 to 2.1.4 (FreeTubeApp#8799)
  Bump babel-loader from 10.1.0 to 10.1.1 in the babel group (FreeTubeApp#8794)
  Bump the webpack group with 2 updates (FreeTubeApp#8797)
  Bump the fortawesome group with 5 updates (FreeTubeApp#8796)
  Bump the eslint group with 3 updates (FreeTubeApp#8795)
  Bump vue from 3.5.29 to 3.5.30 (FreeTubeApp#8800)
  Translated using Weblate (Japanese)
  Serialize the error as a string to pass it back to the main view (FreeTubeApp#8789)
  Bump tar from 7.5.10 to 7.5.11 (FreeTubeApp#8788)
  Workaround for electron/electron#49253 (FreeTubeApp#8714)
  Translated using Weblate (Icelandic)
  Translated using Weblate (Japanese)
  Trim video titles with the local API to avoid breaking the databases (FreeTubeApp#8354)
  Translated using Weblate (Tamil)
  enhance issue templates (FreeTubeApp#8770)
  Bump vue-i18n from 11.2.8 to 11.3.0 (FreeTubeApp#8769)
  ...
PikachuEXE added a commit to PikachuEXE/FreeTube that referenced this pull request Mar 17, 2026
* development: (21 commits)
  Bump webpack-cli from 6.0.1 to 7.0.1 in the webpack group (FreeTubeApp#8816)
  Bump @double-great/stylelint-a11y in the stylelint group (FreeTubeApp#8815)
  Bump youtubei.js from 16.0.1 to 17.0.1 (FreeTubeApp#8818)
  Translated using Weblate (Portuguese)
  Bump electron from 40.8.0 to 41.0.2 (FreeTubeApp#8801)
  Translated using Weblate (German)
  Translated using Weblate (Lithuanian)
  Bump lefthook from 2.1.3 to 2.1.4 (FreeTubeApp#8799)
  Bump babel-loader from 10.1.0 to 10.1.1 in the babel group (FreeTubeApp#8794)
  Bump the webpack group with 2 updates (FreeTubeApp#8797)
  Bump the fortawesome group with 5 updates (FreeTubeApp#8796)
  Bump the eslint group with 3 updates (FreeTubeApp#8795)
  Bump vue from 3.5.29 to 3.5.30 (FreeTubeApp#8800)
  Translated using Weblate (Japanese)
  Serialize the error as a string to pass it back to the main view (FreeTubeApp#8789)
  Bump tar from 7.5.10 to 7.5.11 (FreeTubeApp#8788)
  Workaround for electron/electron#49253 (FreeTubeApp#8714)
  Translated using Weblate (Icelandic)
  Translated using Weblate (Japanese)
  Trim video titles with the local API to avoid breaking the databases (FreeTubeApp#8354)
  ...
Pillowytuba pushed a commit to Pillowytuba/FreeTube that referenced this pull request Mar 19, 2026
* Workaround for electron/electron#49253

* If condition inverted and branches flipped

---------

Co-authored-by: xe16682.contractor <xe16682.contractor@bbva.com>
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.

[Bug]: Application quits itself on Ubuntu after clicking on Show window in Tray menu

5 participants