Skip to content

Commit c4223ce

Browse files
committed
panel: Drop xinerama conversions for panel monitor indices.
With linuxmint/muffin@476ea9b2fadc, monitor order works like it did pre-5.4, primary at 0 and the remaining monitors sorted by connector name following. This lets us remove the translation code we were using to avoid users with existing panel layouts having to re-setup their panels after upgrading.
1 parent 24847d4 commit c4223ce

2 files changed

Lines changed: 4 additions & 60 deletions

File tree

js/ui/cinnamonDBus.js

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -348,8 +348,7 @@ var CinnamonDBus = class {
348348
for (let i = 0; i < Main.layoutManager.monitors.length; i++) {
349349
let current = Main.layoutManager.monitors[i];
350350

351-
let xinerama_index = global.display.logical_index_to_xinerama_index(current.index);
352-
monitors.push(xinerama_index);
351+
monitors.push(current.index);
353352
}
354353
} catch (e) {
355354
log(e.message);
@@ -371,8 +370,7 @@ var CinnamonDBus = class {
371370
throw new Error("GetMonitorWorkRect: invalid monitor index: " + index + ". Must be 0 to " + (n_mons - 1));
372371
}
373372

374-
let logical_index = global.display.xinerama_index_to_logical_index(index);
375-
let rect = global.workspace_manager.get_active_workspace().get_work_area_for_monitor(logical_index);
373+
let rect = global.workspace_manager.get_active_workspace().get_work_area_for_monitor(index);
376374

377375
return [rect.x, rect.y, rect.width, rect.height];
378376
}

js/ui/panel.js

Lines changed: 2 additions & 56 deletions
Original file line numberDiff line numberDiff line change
@@ -235,66 +235,12 @@ function setHeightForPanel(panel) {
235235
return height;
236236
}
237237

238-
function convertSettingsXMonToLMon(strv) {
239-
let out = [];
240-
241-
for (let i = 0; i < strv.length; i++) {
242-
let elements = strv[i].split(":");
243-
244-
if (elements.length !== 3)
245-
continue;
246-
247-
let [id, xmon, pos] = elements;
248-
249-
if (xmon > global.display.get_n_monitors() - 1) {
250-
// log("Skipping panel enabled info for monitor we don't have: " + strv[i]);
251-
out.push(strv[i]);
252-
continue;
253-
}
254-
255-
let l_mon = xmon;
256-
if (!Meta.is_wayland_compositor())
257-
l_mon = global.display.xinerama_index_to_logical_index(xmon);
258-
259-
out.push(`${id}:${l_mon}:${pos}`);
260-
261-
// log(`xmon: ${id}:${xmon}:${pos} to lmon: ${id}:${l_mon}:${pos}`);
262-
}
263-
264-
return out;
265-
}
266-
267-
function convertSettingsLMonToXMon(strv) {
268-
let out = [];
269-
270-
for (let i = 0; i < strv.length; i++) {
271-
let elements = strv[i].split(":");
272-
273-
if (elements.length !== 3)
274-
continue;
275-
276-
let [id, lmon, pos] = elements;
277-
278-
let x_mon = lmon;
279-
if (!Meta.is_wayland_compositor())
280-
x_mon = global.display.logical_index_to_xinerama_index(lmon);
281-
282-
out.push(`${id}:${x_mon}:${pos}`);
283-
284-
// log(`l_mon: ${id}:${l_mon}:${pos} to xmon: ${id}:${x_mon}:${pos}`);
285-
}
286-
287-
return out;
288-
}
289-
290238
function getPanelsEnabledList() {
291-
let panelProperties = global.settings.get_strv("panels-enabled");
292-
return convertSettingsXMonToLMon(panelProperties);
239+
return global.settings.get_strv("panels-enabled");
293240
}
294241

295242
function setPanelsEnabledList(list) {
296-
let converted = convertSettingsLMonToXMon(list)
297-
let panelProperties = global.settings.set_strv("panels-enabled", converted);
243+
global.settings.set_strv("panels-enabled", list);
298244
}
299245

300246
function updatePanelsMeta(meta, panel_props) {

0 commit comments

Comments
 (0)