macOS: mixxx leaves fullscreen view when switching skins

Bug #1777328 reported by Foss-4
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Mixxx
Won't Fix
Medium
Unassigned

Bug Description

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

Tags: macos
Revision history for this message
Benis (beenisss) wrote :

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.

Revision history for this message
Benis (beenisss) wrote :

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.

Changed in mixxx:
status: New → Confirmed
importance: Undecided → Medium
milestone: none → 2.2.0
Revision history for this message
ronso0 (ronso0) wrote :

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.

Revision history for this message
Uwe Klotz (uklotzde-deactivatedaccount) wrote :

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.

Revision history for this message
JosepMa (josepma) wrote :

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.

Revision history for this message
Benis (beenisss) wrote :

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.

Revision history for this message
Benis (beenisss) wrote :

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

Revision history for this message
Daniel Schürmann (daschuer) wrote :

Ups .. Thank you.

Changed in mixxx:
milestone: 2.2.0 → 2.2.1
Revision history for this message
Foss-4 (foss-4) wrote :

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?

Revision history for this message
ronso0 (ronso0) wrote :

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

Revision history for this message
ronso0 (ronso0) wrote :

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

Changed in mixxx:
milestone: 2.2.1 → 2.2.2
Changed in mixxx:
milestone: 2.2.2 → 2.2.3
Changed in mixxx:
milestone: 2.2.3 → 2.2.4
Revision history for this message
Foss-4 (foss-4) wrote :

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.

Revision history for this message
Daniel Schürmann (daschuer) wrote :

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

Revision history for this message
Daniel Schürmann (daschuer) wrote :
Changed in mixxx:
status: Confirmed → Fix Committed
milestone: 2.2.4 → 2.3.0
assignee: nobody → ronso0 (ronso0)
Revision history for this message
Foss-4 (foss-4) wrote :

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)

Changed in mixxx:
status: Fix Committed → In Progress
Revision history for this message
ronso0 (ronso0) wrote :

okay, open for MacOS.
Fixed for Linux in https://github.com/mixxxdj/mixxx/pull/2690

Revision history for this message
Foss-4 (foss-4) wrote :

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

ronso0 (ronso0)
Changed in mixxx:
assignee: ronso0 (ronso0) → nobody
status: In Progress → Confirmed
tags: added: macos
summary: - mixxx leaves fullscreen view when switching skins
+ MacOs: mixxx leaves fullscreen view when switching skins
tags: added: osx
removed: macos
Foss-4 (foss-4)
summary: - MacOs: mixxx leaves fullscreen view when switching skins
+ macOS: mixxx leaves fullscreen view when switching skins
tags: added: macos
removed: osx
Foss-4 (foss-4)
Changed in mixxx:
milestone: 2.3.0 → 2.4.0
Revision history for this message
ronso0 (ronso0) wrote :

@foss-4 Is the issue still in 2.3?

Revision history for this message
Foss-4 (foss-4) wrote :

unchanged and persisting

Revision history for this message
ronso0 (ronso0) wrote :
Changed in mixxx:
assignee: nobody → ronso0 (ronso0)
Revision history for this message
Foss-4 (foss-4) wrote :

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. ❤️

Be (be.ing)
Changed in mixxx:
milestone: 2.4.0 → 2.3.2
Revision history for this message
Foss-4 (foss-4) wrote :

Setting to wontfix as per https://github.com/mixxxdj/mixxx/pull/3859 investigation showing there is no way to address this problem.

Maybe qt6 will behave as expected.

Changed in mixxx:
assignee: ronso0 (ronso0) → nobody
status: Confirmed → Won't Fix
milestone: 2.3.2 → none
Revision history for this message
Swiftb0y (swiftb0y) wrote :

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/9343

lock status: Metadata changes locked and limited to project staff
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.