-
Notifications
You must be signed in to change notification settings - Fork 16
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
feat(Core): migrate engine extensions to rust core (WIP) #411
base: main
Are you sure you want to change the base?
Conversation
…lity. - Restores parity for Udisks2 - Restores parity for SteamRemovableMedia - Fixes a bug with the game loading overlay appearing after OpenGamepadUI is detected by Gamescope. - Properly load devices and profiles in InputPlumber
@@ -59,7 +59,7 @@ var _persist_path: String = "/".join([_data_dir, "launcher.json"]) | |||
var _persist_data: Dictionary = {"version": 1} | |||
var _ogui_window_id := 0 | |||
var should_manage_overlay := true | |||
var logger := Log.get_logger("LaunchManager", Log.LEVEL.INFO) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note for later, fixme
core/systems/launcher/running_app.gd
Outdated
@@ -102,7 +102,7 @@ var steam_close_tries := 0 | |||
## Flag for if OGUI should manage this app. Set to false if app is launched | |||
## outside OGUI and we just want to track it. | |||
var is_ogui_managed: bool = true | |||
var logger := Log.get_logger("RunningApp", Log.LEVEL.INFO) | |||
var logger := Log.get_logger("RunningApp", Log.LEVEL.TRACE) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note for later, fixme
@@ -111,6 +111,7 @@ func _init(item: LibraryLaunchItem, process_id: int, dsp: String) -> void: | |||
display = dsp | |||
|
|||
|
|||
# TODO: Only call this on window creation/deletion |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When is it running now?
- Add input focus change for steam/opengameapdui - Add InputPlumber node to OverlayModeInputManager so it can process dbus signals - Reconnect to all composite devices when InputPlumber starts - Preload all state modules, fixes visibility of overlay... Somehow.
748e8e8
to
5a96942
Compare
logger.debug("Focusable apps changed from", from, "to", to) | ||
# If focusable apps has changed and the currently focused app no longer exists, | ||
# remove the manual focus | ||
var baselayer_app := _xwayland_primary.baselayer_app |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is a base layer app and how is it defined?
@@ -522,7 +522,7 @@ dependencies = [ | |||
[[package]] | |||
name = "gamescope-x11-client" | |||
version = "0.1.0" | |||
source = "git+https://github.com/ShadowBlip/gamescope-x11-client?branch=main#3a0cbe64ba60dffb5ad85f156101c056f764e659" | |||
source = "git+https://github.com/ShadowBlip/gamescope-x11-client?branch=main#deeab5be067bfbb2add2446d8c7fbcbeba7c8c7f" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why did you commit this?
No description provided.