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

overview, viewSelector: Don't use two signals for checking the app button

Browse Source 
Just one will do.
This commit is contained in:
Jasper St. Pierre 2012-10-15 16:36:05 -04:00
parent 7d3ea1ac68
commit 0a8713770b
2 changed files with 11 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
js/ui/overview.js
View File

@ -261,17 +261,16 @@ const Overview = new Lang.Class({
Main.messageTray.show(); Main.messageTray.show();
})); }));
this._viewSelector.connect('apps-button-checked', Lang.bind(this, this._viewSelector.connect('apps-button-checked', Lang.bind(this,
function() { function(vs, checked) {
this.appsActive = true; this.appsActive = checked;
this._thumbnailsBox.hide(); if (checked) {
Main.messageTray.hide(); this._thumbnailsBox.hide();
})); Main.messageTray.hide();
this._viewSelector.connect('apps-button-unchecked', Lang.bind(this, } else {
function() { this._thumbnailsBox.show();
this.appsActive = false; if (this.visible && !this.animationInProgress)
this._thumbnailsBox.show(); Main.messageTray.show();
if (this.visible && !this.animationInProgress) }
Main.messageTray.show();
})); }));
this.connect('app-drag-begin', this.connect('app-drag-begin',

5
js/ui/viewSelector.js
View File

@ -216,10 +216,7 @@ const ViewSelector = new Lang.Class({
}, },
_onShowAppsButtonToggled: function() { _onShowAppsButtonToggled: function() {
if(this._showAppsButton.checked) this.emit('apps-button-checked', this._showAppsButton.checked);
this.emit('apps-button-checked');
else
this.emit('apps-button-unchecked');
if (this.entryNonEmpty) if (this.entryNonEmpty)
this.reset(); this.reset();