Salastil/DankMaterialShell
1
0
Fork 0
mirror of https://github.com/AvengeMedia/DankMaterialShell.git synced 2026-08-02 03:28:28 -04:00
Code Packages Releases Activity

widgets: fix hidden widgets disrupting index when arranging widgets

Browse Source 
fixes #2844
port 1.5

(cherry picked from commit 5dfd875b9e)
This commit is contained in:
bbedward
2026-07-17 21:09:58 -04:00
committed by dms-ci[bot]
parent 9853e1431b
commit f1f528b2cb
1 changed files with 19 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files
quickshell/Modules/Settings/WidgetsTab.qml
+19 -12
View File
@@ -557,20 +557,19 @@ Item {
return sectionId === "left" ? leftSection : sectionId === "center" ? centerSection : sectionId === "right" ? rightSection : null;
}
// Id-based reorder; rebuilds from authoritative objects so every prop (incl. hideWhenIdle) survives
// Id-based reorder; rebuilds from authoritative objects so every prop (incl. hideWhenIdle) survives.
// Consumes one entry per occurrence so duplicate ids (e.g. two spacers) don't collapse, and
// appends unconsumed entries so nothing is silently dropped from the config array.
function reorderSection(sectionId, orderedIds) {
var current = getWidgetsForSection(sectionId);
var byId = {};
current.forEach(w => {
var id = (typeof w === "string" ? w : w.id);
byId[id] = w;
});
var remaining = getWidgetsForSection(sectionId).slice();
var reordered = [];
orderedIds.forEach(id => {
if (byId[id] !== undefined)
reordered.push(byId[id]);
var idx = remaining.findIndex(w => (typeof w === "string" ? w : w.id) === id);
if (idx < 0)
return;
reordered.push(remaining.splice(idx, 1)[0]);
});
setWidgetsForSection(sectionId, reordered);
setWidgetsForSection(sectionId, reordered.concat(remaining));
}
// Move a widget across sections (or within); committed as one atomic bar-config save
@@ -979,8 +978,16 @@ Item {
var isString = typeof widget === "string";
var widgetId = isString ? widget : widget.id;
var widgetDef = baseWidgetDefinitions.find(w => w.id === widgetId);
if (!widgetDef)
return;
if (!widgetDef) {
// Skipping entries would desync row indices from the config array (issue #2844)
widgetDef = {
"id": widgetId,
"text": widgetId || I18n.tr("Unknown"),
"description": "",
"icon": "extension",
"warning": I18n.tr("Unavailable")
};
}
var item = Object.assign({}, widgetDef);
item.enabled = isString ? true : widget.enabled;