[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 11:16:13 +0000 (UTC)
commit fe6f72301c6349b82ec9f6f87441abaab58928bc
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 | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
---
diff --git a/js/ui/shellDBus.js b/js/ui/shellDBus.js
index 469a6c706..28b78a0e4 100644
--- a/js/ui/shellDBus.js
+++ b/js/ui/shellDBus.js
@@ -79,14 +79,15 @@ 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)
+ monitorIndex = this._monitorManager.get_monitor_for_output(connector);
let icon = null;
if (serializedIcon)
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]