remove the ob_root var, its redundant of what Xlib already provides

[chaz/openbox] / openbox / menu.c

diff --git a/openbox/menu.c b/openbox/menu.c
index f7eef7fb08c3e4bcad316bf9fe88a89a79f190d1..aef0356d4bc603e3b447b5ff3dcf19f32580ba20 100644 (file)
--- a/openbox/menu.c
+++ b/openbox/menu.c
@@ -1,6 +1,7 @@
 #include "menu.h"
 #include "openbox.h"
 #include "stacking.h"
+#include "client.h"
 #include "grab.h"
 #include "screen.h"
 #include "geom.h"
@@ -61,7 +62,7 @@ parse_menu_fail:
     g_free(title);
 }
 
-void menu_control_show(Menu *self, int x, int y, Client *client);
+void menu_control_show(Menu *self, int x, int y, ObClient *client);
 
 void menu_destroy_hash_key(Menu *menu)
 {
@@ -211,7 +212,7 @@ Menu *menu_new_full(char *label, char *name, Menu *parent,
 
     attrib.override_redirect = TRUE;
     attrib.event_mask = FRAME_EVENTMASK;
-    self->frame = createWindow(ob_root,
+    self->frame = createWindow(RootWindow(ob_display, ob_screen),
                                CWOverrideRedirect|CWEventMask, &attrib);
     attrib.event_mask = TITLE_EVENTMASK;
     self->title = createWindow(self->frame, CWEventMask, &attrib);
@@ -288,7 +289,7 @@ void menu_add_entry(Menu *menu, MenuEntry *entry)
     g_hash_table_insert(window_map, &entry->item, menu);
 }
 
-void menu_show(char *name, int x, int y, Client *client)
+void menu_show(char *name, int x, int y, ObClient *client)
 {
     Menu *self;
   
@@ -302,7 +303,7 @@ void menu_show(char *name, int x, int y, Client *client)
     menu_show_full(self, x, y, client);
 }  
 
-void menu_show_full(Menu *self, int x, int y, Client *client)
+void menu_show_full(Menu *self, int x, int y, ObClient *client)
 {
     g_assert(self != NULL);
        
@@ -397,14 +398,14 @@ void menu_entry_fire(MenuEntry *self)
    Default menu controller action for showing.
 */
 
-void menu_control_show(Menu *self, int x, int y, Client *client) {
+void menu_control_show(Menu *self, int x, int y, ObClient *client) {
     guint i;
     Rect *a = NULL;
 
     g_assert(!self->invalid);
     
-    for (i = 0; i < screen_num_xin_areas; ++i) {
-        a = screen_physical_area_xinerama(i);
+    for (i = 0; i < screen_num_monitors; ++i) {
+        a = screen_physical_area_monitor(i);
         if (RECT_CONTAINS(*a, x, y))
             break;
     }
@@ -449,7 +450,7 @@ void menu_control_mouseover(MenuEntry *self, gboolean enter) {
            /* need to get the width. is this bad?*/
            menu_render(self->submenu);
 
-            a = screen_physical_area_xinerama(self->parent->xin_area);
+            a = screen_physical_area_monitor(self->parent->xin_area);
 
            if (self->submenu->size.width + x >= a->x + a->width)
                x = self->parent->location.x - self->submenu->size.width -