public-Repo-Mirrors/Top-Bar-Organizer
1
Fork 0
mirror of https://gitlab.gnome.org/julianschacher/top-bar-organizer.git synced 2025-10-27 07:09:07 +00:00
Code Releases Activity

refactor: directly create set of indicator cont. in getValidBoxOrder

Browse Source 
Skip the unecessary intermediate variable and directly create a set.
Also remove the "ToDo: simplify" comment as I don't see how this logic
can be simplified more really.
This commit is contained in:
June 2025-06-12 01:26:44 +02:00
parent 5a09b1a2c8
commit 1e87992081
No known key found for this signature in database
1 changed files with 3 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/extensionModules/BoxOrderManager.ts
View File

@ -259,20 +259,15 @@ export default class BoxOrderManager extends GObject.Object {
// Get a resolved box order.
let resolvedBoxOrder = this.#getResolvedBoxOrder(box);
// ToDo: simplify.
// Get the indicator containers (of the items) currently present in the
// GNOME Shell top bar.
// They should be St.Bins (see link), so ensure that using a filter.
// https://gitlab.gnome.org/GNOME/gnome-shell/-/blob/48.2/js/ui/panelMenu.js?ref_type=tags#L21
const indicatorContainers = [
const indicatorContainers = new Set([
(Main.panel as CustomPanel)._leftBox.get_children(),
(Main.panel as CustomPanel)._centerBox.get_children(),
(Main.panel as CustomPanel)._rightBox.get_children(),
].flat().filter(ic => ic instanceof St.Bin);
// Create an indicator containers set from the indicator containers for
// fast easy access.
const indicatorContainerSet = new Set(indicatorContainers);
].flat().filter(ic => ic instanceof St.Bin));
// Go through the resolved box order and only add items to the valid box
// order, where their indicator is currently present in the GNOME Shell
@ -285,7 +280,7 @@ export default class BoxOrderManager extends GObject.Object {
continue;
}
if (indicatorContainerSet.has(associatedIndicatorContainer)) {
if (indicatorContainers.has(associatedIndicatorContainer)) {
validBoxOrder.push(item);
}
}