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

Fix app starting in fullscreen if a fullscreen video is closed #4649

Conversation

jasonhenriquez
Copy link
Collaborator

@jasonhenriquez jasonhenriquez commented Feb 6, 2024

Fix app starting in fullscreen if a fullscreen video is closed

Pull Request Type

  • Bugfix

Related issue

closes #4534

Description

Co-created with @absidue. Fixes issue of the app starting in fullscreen if a video was closed in fullscreen.

Screenshots

Testing

  • Close a video with Ctrl/Cmd + Q while video is in fullscreen, then restart the app to see app not in fullscreen.
  • Regression: Ensure Implement last window's fullscreen state #2495 is still working correctly (app remains in fullscreen on creating new windows while in fullscreen).

Desktop

  • OS: OpenSUSE Tumbleweed
  • OS Version: 2023xxxx
  • FreeTube version: 0.19.1

Additional context

IIRC, fullscreen works differently in dev mode versus an actual build. Here are the builds, if that helps.

@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Feb 6, 2024
@FreeTubeBot FreeTubeBot enabled auto-merge (squash) February 6, 2024 20:54
@efb4f5ff-1298-471a-8973-3d47447115dc

After upcoming stable release, i will ask if people still experiencing this in #3220

@efb4f5ff-1298-471a-8973-3d47447115dc
Copy link
Member

In both dev mode and with your provided build, im still getting fullscreened after quitting the video player in fullscreen using ctrl + w

idk if it matters but, dev mode is in Ubuntu and build installed on win10

Copy link
Contributor

github-actions bot commented Mar 6, 2024

This PR is stale because it has been open 28 days with no activity. Remove stale label or comment or this will be closed in 14 days.

Copy link
Contributor

github-actions bot commented Apr 4, 2024

This PR is stale because it has been open 28 days with no activity. Remove stale label or comment or this will be closed in 14 days.

@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc added PR: WIP and removed PR: waiting for review For PRs that are complete, tested, and ready for review PR: stale labels Apr 4, 2024
@jasonhenriquez
Copy link
Collaborator Author

If the current fix does not work, I'm really not quite sure what to do.

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

Maybe co creator @absidue has an idea?

auto-merge was automatically disabled May 20, 2024 20:50

Pull request was closed

@github-actions github-actions bot removed the PR: WIP label May 20, 2024
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]: App starts in fullscreen if you used fullscreen before quitting
2 participants