[gnome-shell/wip/carlosg/fix-issue-46: 2/8] keyboard: Remove needless function
- From: Carlos Garnacho <carlosg src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-shell/wip/carlosg/fix-issue-46: 2/8] keyboard: Remove needless function
- Date: Thu, 15 Feb 2018 18:38:24 +0000 (UTC)
commit f8abe073eb5e531fb796ed016735498a78e90886
Author: Carlos Garnacho <carlosg gnome org>
Date: Wed Feb 14 12:46:48 2018 +0100
keyboard: Remove needless function
It boils down to another function call, and has a single caller. Just let
the caller make _setActiveLayer() directly.
https://gitlab.gnome.org/GNOME/gnome-shell/issues/46
js/ui/keyboard.js | 6 +-----
1 file changed, 1 insertion(+), 5 deletions(-)
---
diff --git a/js/ui/keyboard.js b/js/ui/keyboard.js
index 270166997..ad973685d 100644
--- a/js/ui/keyboard.js
+++ b/js/ui/keyboard.js
@@ -813,7 +813,7 @@ var Keyboard = new Lang.Class({
}));
extraButton.connect('released', Lang.bind(this, function() {
if (switchToLevel != null)
- this._onLevelChanged(switchToLevel);
+ this._setActiveLayer(switchToLevel);
else if (keyval != null)
this._keyboardController.keyvalRelease(keyval);
else if (action == 'hide')
@@ -902,10 +902,6 @@ var Keyboard = new Lang.Class({
this.actor.height = maxHeight;
},
- _onLevelChanged: function (level) {
- this._setActiveLayer(level);
- },
-
_onGroupChanged: function () {
this._ensureKeysForGroup(this._keyboardController.getCurrentGroup());
this._setActiveLayer(0);
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]