forked from ValveSoftware/steamos-compositor
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
Support for multiple xwaylands and focus/painting across them #366
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Joshua-Ashton
force-pushed
the
multiple-xwaylands
branch
19 times, most recently
from
January 12, 2022 23:36
c847252
to
dd68d7b
Compare
Joshua-Ashton
force-pushed
the
multiple-xwaylands
branch
8 times, most recently
from
January 19, 2022 00:26
c14b2a3
to
603310e
Compare
Clang gets annoyed when doing offsetof for wl_container_of on a non POD structure -- but there's no virtual inheritance of anything fancy going on so it should shut up.
Doesn't fully support multiple xwaylands or have all the infrastructure for that yet. This just splits the globals out for now and passes the ctx around.
Allows us to bring back the sdlwindow mouse behaviour
Doesn't handle focus or painting anything other than the root context yet.
When the focus changes but the input focus does not (ie. overlay -> steam) the keyboard focus can be left behind on the game, which is not what we want. Track this more ganularly.
Adds --xwayland-count to control the number of xwaylands.
Now we have an entire struct of previous_focus to go off of, we can remove the word current from them.
Makes this easier to read
Makes more sense when reading
…ocus + override determination
More correct, stops reported frametimes being all over the place if steamcompmgr is waiting for stuff to happen.
Joshua-Ashton
force-pushed
the
multiple-xwaylands
branch
from
January 20, 2022 05:55
603310e
to
f095e40
Compare
…ontrol appid is not present inside of xwayland context Allows us to run with multiple xwayland contexts and still have proper focus if the game launches into the first xwayland context
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Allows us to virtualize focus from the game so it never knows we're doing something different.
Closes: #328