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

Mode with combined desktop resolution with Xinerama enabled is unusable with XRandR #930

Closed
SDLBugzilla opened this issue Feb 10, 2021 · 0 comments

Comments

@SDLBugzilla
Copy link
Collaborator

SDLBugzilla commented Feb 10, 2021

This bug report was migrated from our old Bugzilla tracker.

These attachments are available in the static archive:

Reported in version: HG 2.0
Reported for operating system, platform: Linux, All

Comments on the original bug report:

On 2013-06-25 09:52:28 +0000, David Gow wrote:

Created attachment 1196
Remove combined resolution when not using vidmode

On X11, when Xinerama is enabled, it will return a mode (for the display at (0,0)) with the combined resolution of all desktops. (http://hg.libsdl.org/SDL/file/tip/src/video/x11/SDL_x11modes.c#l692 ) While this matches SDL 1.2's behaviour, such a mode does not work correctly with the XRandR backend.

Indeed, on Linux systems, fullscreen modes cannot exceed the resolution of the monitor they are on, as the EWMH fullscreen hint causes a window to cover only a single monitor — or the monitors set by NET_WM_FULLSCREEN_MONITORS, which is not implemented in SDL, and window manager support is a bit spotty.

The solution I have (in the attached patch) is to remove this extra mode (whose presence does not match other OSes) except when using XVidMode, and otherwise replace it with the current video modes used by each monitor, as reported by Xinerama.

-- David

On 2013-07-06 15:33:08 +0000, Sam Lantinga wrote:

Yup, this looks good to me. Thanks!
http://hg.libsdl.org/SDL/rev/9b4430c49235

Feel free to submit other improvements to the X11 mode handling. I admit I'm at the limit of my skill dealing with window managers. :)

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

No branches or pull requests

1 participant