[gnome-shell/benzea/osd-on-connector] shellDBus: Accept connector to restrict OSD to one monitor
- From: Benjamin Berg <bberg src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-shell/benzea/osd-on-connector] shellDBus: Accept connector to restrict OSD to one monitor
- Date: Thu, 11 Apr 2019 13:54:26 +0000 (UTC)
commit 9b0b783d47ed37f0fcb69879044df39a85d9136c
Author: Benjamin Berg <bberg redhat com>
Date: Thu Apr 11 11:43:03 2019 +0200
shellDBus: Accept connector to restrict OSD to one monitor
The monitor index is not something that gnome-shell and
gnome-settings-daemon seem to be agreeing about. Using the connector
string is a much more reliable method of identifying a specific screen
and we are indeed using this already for monitor labling.
So switch over to use the connector rather than the monitor index. If a
user tries to use the old API, then the OSD will simply show up on all
monitors (which is the status quo currently anyway).
https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/489
js/ui/shellDBus.js | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
---
diff --git a/js/ui/shellDBus.js b/js/ui/shellDBus.js
index 469a6c706..b11101c5a 100644
--- a/js/ui/shellDBus.js
+++ b/js/ui/shellDBus.js
@@ -79,14 +79,17 @@ var GnomeShell = class {
for (let param in params)
params[param] = params[param].deep_unpack();
- let { monitor: monitorIndex,
+ let { connector: connector,
label,
level,
max_level: maxLevel,
icon: serializedIcon } = params;
- if (monitorIndex === undefined)
- monitorIndex = -1;
+ let monitorIndex = -1;
+ if (connector) {
+ let monitorManager = Meta.MonitorManager.get();
+ monitorIndex = monitorManager.get_monitor_for_connector(connector);
+ }
let icon = null;
if (serializedIcon)
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]