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

macOS: mixxx leaves fullscreen view when switching skins #9343

Closed
mixxxbot opened this issue Aug 23, 2022 · 23 comments
Closed

macOS: mixxx leaves fullscreen view when switching skins #9343

mixxxbot opened this issue Aug 23, 2022 · 23 comments

Comments

@mixxxbot
Copy link
Collaborator

Reported by: foss-4
Date: 2018-06-17T14:30:10Z
Status: Won't Fix
Importance: Medium
Launchpad Issue: lp1777328
Tags: macos


macOS 10.13.5
mixxx-2.2.0-alpha-pre-master-git6517-release-macintel64.dmg

Reproducible: always

Currently

Switch skins and you'll find yourself in window mode

Expected

stay in fullscreen mode

@mixxxbot
Copy link
Collaborator Author

Commented by: beenisss
Date: 2018-09-02T21:16:53Z


Also on macOS, 10.13.6, and I've seen the same thing both in the official release and building directly from source. The default size is pretty small too.

I've noticed Mixxx resets the window size if you switch skins while not full-screen too, though obviously it's particularly inelegant if you are in full-screen mode at the time.

Additionally, if you quit from full screen, Mixxx reopens with the window at maximum size, but not in a screen of its own. I don't think macOS lets applications open straight to a screen of their own, so all this really does is overwrite the user-determined window size for non-full screen mode. I think the default/recommended behaviour is to always open in a non-maximized window. That's what Safari does at least, which seems a pretty good indicator.

Hope I've explained that properly.

@mixxxbot
Copy link
Collaborator Author

Commented by: beenisss
Date: 2018-09-12T20:26:27Z


When I build the current 2.3.0 alpha-pre it handles this properly. Not sure if that's because it's been fixed or just a quirk of my system, it seems to have a few.

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2018-09-13T12:13:07Z


on Linux it fails with 2.3 as well but differently:
Mixxx stays fullscreen, but the new skin is loaded with window mode size.
Remaining space (at the right & at the bottom stil shows the framebuffer from the previous skin. Resizing, or toggling fullscreen twice paints the skin correctly.

@mixxxbot
Copy link
Collaborator Author

Commented by: uklotzde
Date: 2018-10-21T20:56:10Z


2.2 + XWayland + Full-screen view: Ok
You can watch the repainting while switching skins, but eventually the new skin fills the whole window/screen. I don't need to toggle full-screen view again.

2.2 + XWayland + Window view: (Ok)
The application window is resized to minimum size when switching skins. But I don't consider this a bug, just a minor glitch.

@mixxxbot
Copy link
Collaborator Author

Commented by: JosepMaJAZ
Date: 2018-10-21T21:06:18Z


Windows 10: Mixxx 2.2 built from sources this weekend. Things works as expected as in:

When in fullscreen and switching skins, it temporarily switches to windowed mode, and when the skin is loaded it goes back into fullscreen.

Could it be that this "switching on/off" of the fullscreen mode causes it to not enter fullscreen in some circumstances?

What I've observed in the other bug at #⁠1773587 is that when used in maximized state it does what is reported here. But maximized is not the same as fullscreen.

@mixxxbot
Copy link
Collaborator Author

Commented by: beenisss
Date: 2018-10-22T19:09:21Z


I said this in an earlier comment:

When I build the current 2.3.0 alpha-pre it handles this properly. Not sure if that's because it's been fixed or just a quirk of my system, it seems to have a few.

I suspect I mis-stated the case. I currently get the issue with all builds and releases.

The only thing different is that the official release doesn't restore to a maximized window properly, whereas the beta and any build from a recent master do.

@mixxxbot
Copy link
Collaborator Author

Commented by: beenisss
Date: 2018-12-25T14:06:06Z


I don't think this is the same issue as #⁠1773587

@mixxxbot
Copy link
Collaborator Author

Commented by: daschuer
Date: 2018-12-25T22:53:22Z


Ups .. Thank you.

@mixxxbot
Copy link
Collaborator Author

Commented by: foss-4
Date: 2019-03-02T00:10:25Z


Is this issue on the radar for 2.2.1 indeed? Is there a PR already? Is there a dev willing to work on this?

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2019-03-05T16:37:41Z


btw: skins are also shown at minimal size after (re)loading skins in windowed mode although Mixxx fills the screen.

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2019-03-05T22:33:28Z


another observation (ubuntu studio, X11):
window with custom size > maximize = maximized window
window maximized > F11 = fullscreen
fullscreen > F11 = window with custom size

window with custom size > reload skin = window at minimal size (set by skin)

some hopefully helpful input before I work on it myself:
https://stackoverflow.com/questions/38182367/how-to-toggle-a-window-from-fullscreen-to-normal-mode-programmatically#⁠38182513

maybe we should do this when reloading the main window:

  • if fullscreen: leave fullscreen, restore minimal size (un-maximize)
  • if maximized: restore minimal size (un-maximize), store maximize state
    (* not fullscreen, not maximized: store window size (as it does now))
    X reload skin
    (should be at normal size now)
  • enter fullscreen mode or restore maximize state

@mixxxbot
Copy link
Collaborator Author

Commented by: foss-4
Date: 2020-03-04T13:07:15Z


This problem is unchanged. Is anybody willing to look into this? Seems like various OS are affected. Functionality is not decreased but it's somewhat annoying to have windows jump around and switch size by themselves.

@mixxxbot
Copy link
Collaborator Author

Commented by: daschuer
Date: 2020-03-04T15:15:04Z


I can confirm this.
Unfortunately no one had found time to look into it yet.

@mixxxbot
Copy link
Collaborator Author

Commented by: daschuer
Date: 2020-04-30T06:25:28Z


This was fixed here
#2690

@mixxxbot
Copy link
Collaborator Author

Commented by: foss-4
Date: 2020-04-30T20:25:44Z


Sorry, not fixed on macOS. Still leaving full-screen when switching skin.
macOS 10.15.4
mixxx 2.3.0-alpha-pre (build master r7331)

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2020-04-30T22:06:38Z


okay, open for MacOS.
Fixed for Linux in #2690

@mixxxbot
Copy link
Collaborator Author

Commented by: foss-4
Date: 2020-04-30T22:17:49Z


yup, original report was filed against macOS so let's leave this bug open.

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2021-05-08T20:58:37Z


@foss-4 Is the issue still in 2.3?

@mixxxbot
Copy link
Collaborator Author

Commented by: foss-4
Date: 2021-05-08T23:36:20Z


unchanged and persisting

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2021-05-14T01:33:38Z


maybe fixed by
#3859

@mixxxbot
Copy link
Collaborator Author

Commented by: foss-4
Date: 2021-05-14T11:32:37Z


Goes back to window mode after skin switch testing 2.3-beta-3983-g674fc8ceee (HEAD branch) (from this PR) on macOS 11.3.1.

Oh and thanks very much for looking into this. ❤️

@mixxxbot
Copy link
Collaborator Author

Commented by: foss-4
Date: 2021-12-05T11:21:26Z


Setting to wontfix as per #3859 investigation showing there is no way to address this problem.

Maybe qt6 will behave as expected.

@mixxxbot
Copy link
Collaborator Author

Issue closed with status Won't Fix.

@mixxxbot mixxxbot transferred this issue from another repository Aug 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant