[gnome-shell] system: Don't execute Settings
- From: Georges Basile Stavracas Neto <gbsneto src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-shell] system: Don't execute Settings
- Date: Tue, 31 Jul 2018 16:33:29 +0000 (UTC)
commit f433b12d6e7110031a46f74b83d3318b1f242fd8
Author: Georges Basile Stavracas Neto <georges stavracas gmail com>
Date: Tue Jul 31 13:23:05 2018 -0300
system: Don't execute Settings
In a9ad91c831, a bug was introduced in the following code:
```c
this._settingsAction.connect('clicked',
this._onSettingsClicked().bind(this));
```
Notice that the callback is being executed! This commit
fixes that by removing the '()' from the callback.
js/ui/status/system.js | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
---
diff --git a/js/ui/status/system.js b/js/ui/status/system.js
index 81b985c1f..c695f37be 100644
--- a/js/ui/status/system.js
+++ b/js/ui/status/system.js
@@ -269,7 +269,7 @@ var Indicator = new Lang.Class({
app.get_name()];
this._settingsAction = this._createActionButton(icon, name);
this._settingsAction.connect('clicked',
- this._onSettingsClicked().bind(this));
+ this._onSettingsClicked.bind(this));
} else {
log('Missing required core component Settings, expect troubleā¦');
this._settingsAction = new St.Widget();
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]