brl/gnome-shell
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

padOsd: Move all coord/existence checks to _addLabel()

Browse Source 
Drops some repetitive code. Also rely completely on the label/leader
elements being found in order to find buttons/rings/strips.

https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/1290
This commit is contained in:
Carlos Garnacho 2020-05-29 15:43:55 +02:00 committed by Florian Müllner
parent dac05c7e53
commit c511c469fe
1 changed files with 13 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
js/ui/padOsd.js
View File

@ -345,29 +345,22 @@ var PadDiagram = GObject.registerClass({
}
_initLabels() {
// FIXME: Fix num buttons.
let i = 0;
for (i = 0; i < 50; i++) {
let [found] = this._getLabelCoords(Meta.PadActionType.BUTTON, i);
if (!found)
for (i = 0; ; i++) {
if (!this._addLabel(Meta.PadActionType.BUTTON, i))
break;
this._addLabel(Meta.PadActionType.BUTTON, i);
}
for (i = 0; i < 2; i++) {
let [found] = this._getLabelCoords(Meta.PadActionType.RING, i, CW);
if (!found)
for (i = 0; ; i++) {
if (!this._addLabel(Meta.PadActionType.RING, i, CW) ||
!this._addLabel(Meta.PadActionType.RING, i, CCW))
break;
this._addLabel(Meta.PadActionType.RING, i, CW);
this._addLabel(Meta.PadActionType.RING, i, CCW);
}
for (i = 0; i < 2; i++) {
let [found] = this._getLabelCoords(Meta.PadActionType.STRIP, i, UP);
if (!found)
for (i = 0; ; i++) {
if (!this._addLabel(Meta.PadActionType.STRIP, i, UP) ||
!this._addLabel(Meta.PadActionType.STRIP, i, DOWN))
break;
this._addLabel(Meta.PadActionType.STRIP, i, UP);
this._addLabel(Meta.PadActionType.STRIP, i, DOWN);
}
}
@ -583,9 +576,14 @@ var PadDiagram = GObject.registerClass({
}
_addLabel(type, idx, dir) {
let [found] = this._getLabelCoords(type, idx, dir);
if (!found)
return false;
let label = new St.Label();
this._labels.push([label, type, idx, dir]);
this.add_actor(label);
return true;
}
updateLabels(getText) {