Workaround for https://github.com/electron/electron/issues/49253#8714
Conversation
|
Could you please invert the check and flip the if-else branches as it is a bit more readable that way. |
|
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: |
|
Please make sure you are using Node.js 24+. |
Head branch was pushed to by a user without write access
|
Thanks, that did the trick, the PR is updated. |
There was a problem hiding this comment.
@Devenor idk if you can do something about it but it looks very glitchy
VirtualBoxVM_yhY87kgjlE.mp4
|
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 ? |
|
@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). |
This was turned on in the clip
This was turned off in the clip
Will try |
|
@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. |
efb4f5ff-1298-471a-8973-3d47447115dc
left a comment
There was a problem hiding this comment.
I think its safe to say that these issues are unrelated to your PR. LGTM. Thanks @Devenor !
* 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)
* 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) ...
* 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) ...
* Workaround for electron/electron#49253 * If condition inverted and branches flipped --------- Co-authored-by: xe16682.contractor <xe16682.contractor@bbva.com>
Pull Request Type
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
Desktop
Additional context