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

Do not set compositor bypass hint for SDL Window #216

Closed

Conversation

nigels-com
Copy link
Contributor

I happen to be trying out the KDE desktop on Ubuntu 18.04 today.

SDL2 sets a hint that disables the KDE compositor which results in desktop glitching at startup and shutdown. I found this a poor experience, but possibly what you'd want on a slow/cheap system or in fullscreen mode, to maximise performance.

This code change works for me. The alternative in KDE is to disable this globally or whitelist the application to specifically ignore the hint.

Screenshot_20191201_220203

@irmen
Copy link
Contributor

irmen commented Jan 5, 2021

Ah, this was bothering me as well, now I know what is causing the glitching/flickering.

@indigodarkwolf
Copy link
Collaborator

Necro bump.

@mist64 Were there any issues discovered with this, that this wouldn't be merged in? Seems harmless enough, and apparently fixes a nuisance for Linux users.

@greg-king5
Copy link

Were there any issues discovered with this, that this wouldn't be merged in? Seems harmless enough, and apparently fixes a nuisance for Linux users.

The emulator's code has changed. There are conflicts. That patch must be rebased on the current code.

@indigodarkwolf
Copy link
Collaborator

Well... that's a ridiculous failure to merge. :P And I don't have write permissions to the branch/PR. Guess I'll submit a new one.

@irmen
Copy link
Contributor

irmen commented Apr 5, 2021

i suppose this one can be abandoned now since the other PR has been merged!

@indigodarkwolf
Copy link
Collaborator

Indeed. @mist64, can you close this? The original author seems to be a ghost now.

@mist64 mist64 closed this Apr 6, 2021
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.

None yet

5 participants