Skip to content

Workaround ready-to-show not firing consistently on wayland - #8294

Merged
FreeTubeBot merged 1 commit into
FreeTubeApp:developmentfrom
absidue:wayland-ready-to-show
Nov 18, 2025
Merged

Workaround ready-to-show not firing consistently on wayland#8294
FreeTubeBot merged 1 commit into
FreeTubeApp:developmentfrom
absidue:wayland-ready-to-show

Conversation

@absidue

@absidue absidue commented Nov 16, 2025

Copy link
Copy Markdown
Member

Pull Request Type

  • Bugfix

Related issue

Description

This pull request works around the ready-to-show event not firing consistently on wayland, which also seems to affect FreeTube as @efb4f5ff-1298-471a-8973-3d47447115dc mentioned that the window wasn't showing up for them, by listening for the did-finish-load event on wayland.

Testing

I tested that this works on Windows by hardcoding the check to true or false but I don't have a machine running Linux with wayland, so I wasn't able to test if the wayland detection is implemented properly.

Desktop

  • OS: Windows
  • OS Version: 11

@github-actions github-actions Bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Nov 16, 2025
@FreeTubeBot
FreeTubeBot enabled auto-merge (squash) November 16, 2025 14:22
@FreeTubeBot
FreeTubeBot merged commit f1d4991 into FreeTubeApp:development Nov 18, 2025
5 checks passed
@github-actions github-actions Bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Nov 18, 2025
PikachuEXE added a commit to PikachuEXE/FreeTube that referenced this pull request Nov 18, 2025
* development: (22 commits)
  Workaround ready-to-show not firing consistently on wayland (FreeTubeApp#8294)
  Bump electron from 38.4.0 to 39.2.1 (FreeTubeApp#8303)
  Migrate the WatchVideoInfo component to the composition API (FreeTubeApp#8295)
  Translated using Weblate (Hungarian)
  Translated using Weblate (Norwegian Bokmål)
  Translated using Weblate (French)
  Bump eslint-plugin-jsdoc from 61.1.12 to 61.2.1 in the eslint group (FreeTubeApp#8298)
  Replace broken trending with a local API only equivalent (FreeTubeApp#8289)
  Fix settings layout issues after the Vue 3 migration (FreeTubeApp#8292)
  Bump lefthook from 2.0.3 to 2.0.4 (FreeTubeApp#8299)
  Bump shaka-player from 4.16.8 to 4.16.9 (FreeTubeApp#8301)
  Bump electron-builder from 26.1.0 to 26.2.0 (FreeTubeApp#8302)
  Fix playlist video view counts on the local API (FreeTubeApp#8290)
  change hardcoded color app text logoColor (FreeTubeApp#8288)
  Translated using Weblate (Russian)
  Translated using Weblate (German)
  wrap long playlist strings (FreeTubeApp#8275)
  Translated using Weblate (Bulgarian)
  Bump js-yaml from 4.1.0 to 4.1.1 (FreeTubeApp#8284)
  bump workflows node-version to 24 (FreeTubeApp#8279)
  ...

# Conflicts:
#	src/renderer/helpers/api/invidious.js
PikachuEXE added a commit to PikachuEXE/FreeTube that referenced this pull request Nov 18, 2025
* development:
  Translated using Weblate (Chinese (Simplified Han script))
  Workaround ready-to-show not firing consistently on wayland (FreeTubeApp#8294)
  Bump electron from 38.4.0 to 39.2.1 (FreeTubeApp#8303)
  Migrate the WatchVideoInfo component to the composition API (FreeTubeApp#8295)
  Translated using Weblate (Hungarian)
  Translated using Weblate (Norwegian Bokmål)
  Translated using Weblate (French)
  Bump eslint-plugin-jsdoc from 61.1.12 to 61.2.1 in the eslint group (FreeTubeApp#8298)
  Replace broken trending with a local API only equivalent (FreeTubeApp#8289)
  Fix settings layout issues after the Vue 3 migration (FreeTubeApp#8292)
  Bump lefthook from 2.0.3 to 2.0.4 (FreeTubeApp#8299)
  Bump shaka-player from 4.16.8 to 4.16.9 (FreeTubeApp#8301)
  Bump electron-builder from 26.1.0 to 26.2.0 (FreeTubeApp#8302)

# Conflicts:
#	src/renderer/helpers/api/local.js
@absidue
absidue deleted the wayland-ready-to-show branch November 18, 2025 06:08
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