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

Remove workarounds for mutual imports problems

Browse Source 
Now that gjs Bug #558741 is fixed, we can import Main directly from
the toplevel of overlay.js/panel.js without causing problems.

svn path=/trunk/; revision=41
This commit is contained in:
Owen Taylor 2008-11-09 18:01:59 +00:00
parent 6a0c6a5df6
commit 536bc24f61
2 changed files with 2 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
js/ui/overlay.js
View File

@ -5,6 +5,7 @@ const Meta = imports.gi.Meta;
const Shell = imports.gi.Shell;
const Tweener = imports.tweener.tweener;
const Main = imports.ui.main;
const Panel = imports.ui.panel;
const OVERLAY_BACKGROUND_COLOR = new Clutter.Color();
@ -228,7 +229,6 @@ Overlay.prototype = {
},
_deactivate : function() {
let Main = imports.ui.main;
Main.hide_overlay();
}
};

8
js/ui/panel.js
View File

@ -5,10 +5,7 @@ const Mainloop = imports.mainloop;
const Shell = imports.gi.Shell;
const Clutter = imports.gi.Clutter;
// The mutual import here causes things to break in weird ways,
// (http://bugzilla.gnome.org/show_bug.cgi?id=558741)
// So we do a local import below
// const Main = imports.ui.main;
const Main = imports.ui.main;
const PANEL_HEIGHT = 32;
const PANEL_BACKGROUND_COLOR = new Clutter.Color();
@ -46,9 +43,6 @@ Panel.prototype = {
message.connect('button-press-event',
function(o, event) {
// See comment above
let Main = imports.ui.main;
if (Main.overlay.visible)
Main.hide_overlay();
else