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

chrome: Make affectsStruts default to false

Browse Source 
Since we only want it to be true for the panel, and nothing else.

https://bugzilla.gnome.org/show_bug.cgi?id=655813
This commit is contained in:
Dan Winship 2011-07-21 10:18:15 -04:00
parent dbeab0ef87
commit a376cd1610
5 changed files with 16 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
js/ui/chrome.js
View File

@ -16,7 +16,7 @@ const ScreenSaver = imports.misc.screenSaver;
const defaultParams = { const defaultParams = {
visibleInFullscreen: false, visibleInFullscreen: false,
affectsStruts: true, affectsStruts: false,
affectsInputRegion: true affectsInputRegion: true
}; };
@ -71,19 +71,18 @@ Chrome.prototype = {
// @actor: an actor to add to the chrome layer // @actor: an actor to add to the chrome layer
// @params: (optional) additional params // @params: (optional) additional params
// //
// Adds @actor to the chrome layer and extends the input region // Adds @actor to the chrome layer and (unless %affectsInputRegion
// and window manager struts to include it. (Window manager struts // is %false) extends the input region to include it. Changes in
// will only be affected if @actor is touching a screen edge.) // @actor's size, position, and visibility will automatically
// Changes in @actor's size and position will automatically result // result in appropriate changes to the input region.
// in appropriate changes to the input region and struts. Changes //
// in its visibility will affect the input region, but NOT the // If %affectsStruts is %true (and @actor is along a screen edge),
// struts. // then @actor's size and position will also affect the window
// manager struts. Changes to @actor's visibility will NOT affect
// whether or not the strut is present, however.
// //
// If %visibleInFullscreen is %true, the actor will be visible // If %visibleInFullscreen is %true, the actor will be visible
// even when a fullscreen window should be covering it. // even when a fullscreen window should be covering it.
//
// If %affectsStruts or %affectsInputRegion is %false, the actor
// will not have the indicated effect.
addActor: function(actor, params) { addActor: function(actor, params) {
this.actor.add_actor(actor); this.actor.add_actor(actor);
this._trackActor(actor, params); this._trackActor(actor, params);

2
js/ui/layout.js
View File

@ -109,7 +109,7 @@ LayoutManager.prototype = {
let corner = new HotCorner(); let corner = new HotCorner();
this._hotCorners.push(corner); this._hotCorners.push(corner);
corner.actor.set_position(cornerX, cornerY); corner.actor.set_position(cornerX, cornerY);
Main.chrome.addActor(corner.actor, { affectsStruts: false }); Main.chrome.addActor(corner.actor);
} }
}, },

3
js/ui/messageTray.js
View File

@ -1330,8 +1330,7 @@ MessageTray.prototype = {
this._notificationRemoved = false; this._notificationRemoved = false;
this._reNotifyAfterHideNotification = null; this._reNotifyAfterHideNotification = null;
Main.chrome.addActor(this.actor, { affectsStruts: false, Main.chrome.addActor(this.actor, { visibleInFullscreen: true });
visibleInFullscreen: true });
Main.chrome.trackActor(this._notificationBin); Main.chrome.trackActor(this._notificationBin);
Main.chrome.trackActor(this._summaryBoxPointer.actor); Main.chrome.trackActor(this._summaryBoxPointer.actor);

8
js/ui/panel.js
View File

@ -941,11 +941,9 @@ Panel.prototype = {
Main.statusIconDispatcher.connect('status-icon-added', Lang.bind(this, this._onTrayIconAdded)); Main.statusIconDispatcher.connect('status-icon-added', Lang.bind(this, this._onTrayIconAdded));
Main.statusIconDispatcher.connect('status-icon-removed', Lang.bind(this, this._onTrayIconRemoved)); Main.statusIconDispatcher.connect('status-icon-removed', Lang.bind(this, this._onTrayIconRemoved));
Main.chrome.addActor(this.actor); Main.chrome.addActor(this.actor, { affectsStruts: true });
Main.chrome.addActor(this._leftCorner.actor, { affectsStruts: false, Main.chrome.addActor(this._leftCorner.actor, { affectsInputRegion: false });
affectsInputRegion: false }); Main.chrome.addActor(this._rightCorner.actor, { affectsInputRegion: false });
Main.chrome.addActor(this._rightCorner.actor, { affectsStruts: false,
affectsInputRegion: false });
Main.ctrlAltTabManager.addGroup(this.actor, _("Top Bar"), 'start-here', Main.ctrlAltTabManager.addGroup(this.actor, _("Top Bar"), 'start-here',
{ sortGroup: CtrlAltTab.SortGroup.TOP }); { sortGroup: CtrlAltTab.SortGroup.TOP });

2
js/ui/panelMenu.js
View File

@ -27,7 +27,7 @@ Button.prototype = {
this.menu.actor.add_style_class_name('panel-menu'); this.menu.actor.add_style_class_name('panel-menu');
this.menu.connect('open-state-changed', Lang.bind(this, this._onOpenStateChanged)); this.menu.connect('open-state-changed', Lang.bind(this, this._onOpenStateChanged));
this.menu.actor.connect('key-press-event', Lang.bind(this, this._onMenuKeyPress)); this.menu.actor.connect('key-press-event', Lang.bind(this, this._onMenuKeyPress));
Main.chrome.addActor(this.menu.actor, { affectsStruts: false }); Main.chrome.addActor(this.menu.actor);
this.menu.actor.hide(); this.menu.actor.hide();
}, },