X-Git-Url: https://git.dogcows.com/gitweb?a=blobdiff_plain;f=openbox%2Fdock.c;h=8cb40fb6c7ca6614c65d95272bc284fada96fbd4;hb=355981bf14a888b310be40018f9b7d35bc05e606;hp=d46f76745cb33076a8c022bd92af17432fd451fb;hpb=e39762b05b87e9f3d125450a1a27618f52de1354;p=chaz%2Fopenbox

diff --git a/openbox/dock.c b/openbox/dock.c
index d46f7674..8cb40fb6 100644
--- a/openbox/dock.c
+++ b/openbox/dock.c
@@ -1,4 +1,25 @@
+/* -*- indent-tabs-mode: nil; tab-width: 4; c-basic-offset: 4; -*-
+
+   dock.c for the Openbox window manager
+   Copyright (c) 2006        Mikael Magnusson
+   Copyright (c) 2003-2007   Dana Jansens
+
+   This program is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 2 of the License, or
+   (at your option) any later version.
+
+   This program is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
+
+   See the COPYING file for a copy of the GNU General Public License.
+*/
+
+#include "debug.h"
 #include "dock.h"
+#include "mainloop.h"
 #include "screen.h"
 #include "prop.h"
 #include "config.h"
@@ -10,39 +31,89 @@
                          EnterWindowMask | LeaveWindowMask)
 #define DOCKAPP_EVENT_MASK (StructureNotifyMask)
 
-static Dock *dock;
+static ObDock *dock;
 
-Strut dock_strut;
+StrutPartial dock_strut;
 
-void dock_startup()
+static void dock_app_grab_button(ObDockApp *app, gboolean grab)
+{
+    if (grab) {
+        grab_button_full(config_dock_app_move_button,
+                         config_dock_app_move_modifiers, app->icon_win,
+                         ButtonPressMask | ButtonReleaseMask |
+                         ButtonMotionMask,
+                         GrabModeAsync, OB_CURSOR_MOVE);
+    } else {
+        ungrab_button(config_dock_app_move_button,
+                      config_dock_app_move_modifiers, app->icon_win);
+    }
+}
+
+void dock_startup(gboolean reconfig)
 {
     XSetWindowAttributes attrib;
 
-    STRUT_SET(dock_strut, 0, 0, 0, 0);
+    if (reconfig) {
+        GList *it;
+
+        XSetWindowBorder(ob_display, dock->frame,
+                         RrColorPixel(ob_rr_theme->osd_border_color));
+        XSetWindowBorderWidth(ob_display, dock->frame, ob_rr_theme->obwidth);
+
+        RrAppearanceFree(dock->a_frame);
+        dock->a_frame = RrAppearanceCopy(ob_rr_theme->osd_hilite_bg);
+
+        stacking_add(DOCK_AS_WINDOW(dock));
+
+        dock_configure();
+        dock_hide(TRUE);
 
-    dock = g_new0(struct Dock, 1);
+        for (it = dock->dock_apps; it; it = g_list_next(it))
+            dock_app_grab_button(it->data, TRUE);
+        return;
+    }
+
+    STRUT_PARTIAL_SET(dock_strut, 0, 0, 0, 0,
+                      0, 0, 0, 0, 0, 0, 0, 0);
+
+    dock = g_new0(ObDock, 1);
     dock->obwin.type = Window_Dock;
 
     dock->hidden = TRUE;
 
     attrib.event_mask = DOCK_EVENT_MASK;
     attrib.override_redirect = True;
-    dock->frame = XCreateWindow(ob_display, ob_root, 0, 0, 1, 1, 0,
+    dock->frame = XCreateWindow(ob_display, RootWindow(ob_display, ob_screen),
+                                0, 0, 1, 1, 0,
                                 RrDepth(ob_rr_inst), InputOutput,
                                 RrVisual(ob_rr_inst),
                                 CWOverrideRedirect | CWEventMask,
                                 &attrib);
-    dock->a_frame = RrAppearanceCopy(ob_rr_theme->a_unfocused_title);
-    XSetWindowBorder(ob_display, dock->frame, ob_rr_theme->b_color->pixel);
-    XSetWindowBorderWidth(ob_display, dock->frame, ob_rr_theme->bwidth);
+    dock->a_frame = RrAppearanceCopy(ob_rr_theme->osd_hilite_bg);
+    XSetWindowBorder(ob_display, dock->frame,
+                     RrColorPixel(ob_rr_theme->osd_border_color));
+    XSetWindowBorderWidth(ob_display, dock->frame, ob_rr_theme->obwidth);
+
+    /* Setting the window type so xcompmgr can tell what it is */
+    PROP_SET32(dock->frame, net_wm_window_type, atom,
+               prop_atoms.net_wm_window_type_dock);
 
     g_hash_table_insert(window_map, &dock->frame, dock);
     stacking_add(DOCK_AS_WINDOW(dock));
-    stacking_raise(DOCK_AS_WINDOW(dock));
 }
 
-void dock_shutdown()
+void dock_shutdown(gboolean reconfig)
 {
+    if (reconfig) {
+        GList *it;
+
+        stacking_remove(DOCK_AS_WINDOW(dock));
+
+        for (it = dock->dock_apps; it; it = g_list_next(it))
+            dock_app_grab_button(it->data, FALSE);
+        return;
+    }
+
     XDestroyWindow(ob_display, dock->frame);
     RrAppearanceFree(dock->a_frame);
     g_hash_table_remove(window_map, &dock->frame);
@@ -51,11 +122,11 @@ void dock_shutdown()
 
 void dock_add(Window win, XWMHints *wmhints)
 {
-    DockApp *app;
+    ObDockApp *app;
     XWindowAttributes attrib;
-    char **data;
+    gchar **data;
 
-    app = g_new0(DockApp, 1);
+    app = g_new0(ObDockApp, 1);
     app->obwin.type = Window_DockApp;
     app->win = win;
     app->icon_win = (wmhints->flags & IconWindowHint) ?
@@ -63,7 +134,7 @@ void dock_add(Window win, XWMHints *wmhints)
 
     if (PROP_GETSS(app->win, wm_class, locale, &data)) {
         if (data[0]) {
-	    app->name = g_strdup(data[0]);
+            app->name = g_strdup(data[0]);
             if (data[1])
                 app->class = g_strdup(data[1]);
         }
@@ -91,8 +162,8 @@ void dock_add(Window win, XWMHints *wmhints)
       member set the root window, and one set to the client, but both get
       handled and need to be ignored.
     */
-    if (ob_state == State_Starting)
-	app->ignore_unmaps += 2;
+    if (ob_state() == OB_STATE_STARTING)
+        app->ignore_unmaps += 2;
 
     if (app->win != app->icon_win) {
         /* have to map it so that it can be re-managed on a restart */
@@ -107,13 +178,11 @@ void dock_add(Window win, XWMHints *wmhints)
     XChangeSaveSet(ob_display, app->icon_win, SetModeInsert);
     XSelectInput(ob_display, app->icon_win, DOCKAPP_EVENT_MASK);
 
-    grab_button_full(2, 0, app->icon_win,
-                     ButtonPressMask | ButtonReleaseMask | ButtonMotionMask,
-                     GrabModeAsync, ob_cursors.move);
+    dock_app_grab_button(app, TRUE);
 
     g_hash_table_insert(window_map, &app->icon_win, app);
 
-    g_message("Managed Dock App: 0x%lx (%s)", app->icon_win, app->class);
+    ob_debug("Managed Dock App: 0x%lx (%s)\n", app->icon_win, app->class);
 }
 
 void dock_remove_all()
@@ -122,9 +191,9 @@ void dock_remove_all()
         dock_remove(dock->dock_apps->data, TRUE);
 }
 
-void dock_remove(DockApp *app, gboolean reparent)
+void dock_remove(ObDockApp *app, gboolean reparent)
 {
-    ungrab_button(2, 0, app->icon_win);
+    dock_app_grab_button(app, FALSE);
     XSelectInput(ob_display, app->icon_win, NoEventMask);
     /* remove the window from our save set */
     XChangeSaveSet(ob_display, app->icon_win, SetModeDelete);
@@ -133,12 +202,13 @@ void dock_remove(DockApp *app, gboolean reparent)
     g_hash_table_remove(window_map, &app->icon_win);
 
     if (reparent)
-	XReparentWindow(ob_display, app->icon_win, ob_root, app->x, app->y);
+        XReparentWindow(ob_display, app->icon_win,
+                        RootWindow(ob_display, ob_screen), app->x, app->y);
 
     dock->dock_apps = g_list_remove(dock->dock_apps, app);
     dock_configure();
 
-    g_message("Unmanaged Dock App: 0x%lx (%s)", app->icon_win, app->class);
+    ob_debug("Unmanaged Dock App: 0x%lx (%s)\n", app->icon_win, app->class);
 
     g_free(app->name);
     g_free(app->class);
@@ -148,278 +218,379 @@ void dock_remove(DockApp *app, gboolean reparent)
 void dock_configure()
 {
     GList *it;
-    int spot;
-    int gravity;
-    int minw, minh;
+    gint hspot, vspot;
+    gint gravity;
+    gint l, r, t, b;
+    gint strw, strh;
+    Rect *a;
 
-    RrMinsize(dock->a_frame, &minw, &minh);
+    RrMargins(dock->a_frame, &l, &t, &r, &b);
 
-    dock->w = dock->h = 0;
+    dock->area.width = dock->area.height = 0;
 
     /* get the size */
-    for (it = dock->dock_apps; it; it = it->next) {
-        struct DockApp *app = it->data;
-        if (config_dock_horz) {
-            dock->w += app->w;
-            dock->h = MAX(dock->h, app->h);
-        } else {
-            dock->w = MAX(dock->w, app->w);
-            dock->h += app->h;
+    for (it = dock->dock_apps; it; it = g_list_next(it)) {
+        ObDockApp *app = it->data;
+        switch (config_dock_orient) {
+        case OB_ORIENTATION_HORZ:
+            dock->area.width += app->w;
+            dock->area.height = MAX(dock->area.height, app->h);
+            break;
+        case OB_ORIENTATION_VERT:
+            dock->area.width = MAX(dock->area.width, app->w);
+            dock->area.height += app->h;
+            break;
         }
     }
 
-    spot = (config_dock_horz ? minw : minh) / 2;
+    dock->area.width += l + r;
+    dock->area.height += t + b;
+
+    hspot = l;
+    vspot = t;
 
     /* position the apps */
-    for (it = dock->dock_apps; it; it = it->next) {
-        struct DockApp *app = it->data;
-        if (config_dock_horz) {
-            app->x = spot;
-            app->y = (dock->h - app->h) / 2;
-            spot += app->w;
-        } else {
-            app->x = (dock->w - app->w) / 2;
-            app->y = spot;
-            spot += app->h;
+    for (it = dock->dock_apps; it; it = g_list_next(it)) {
+        ObDockApp *app = it->data;
+        switch (config_dock_orient) {
+        case OB_ORIENTATION_HORZ:
+            app->x = hspot;
+            app->y = (dock->area.height - app->h) / 2;
+            hspot += app->w;
+            break;
+        case OB_ORIENTATION_VERT:
+            app->x = (dock->area.width - app->w) / 2;
+            app->y = vspot;
+            vspot += app->h;
+            break;
         }
 
         XMoveWindow(ob_display, app->icon_win, app->x, app->y);
     }
 
     /* used for calculating offsets */
-    dock->w += ob_rr_theme->bwidth * 2;
-    dock->h += ob_rr_theme->bwidth * 2;
+    dock->area.width += ob_rr_theme->obwidth * 2;
+    dock->area.height += ob_rr_theme->obwidth * 2;
+
+    a = screen_physical_area_all_monitors();
 
     /* calculate position */
-    switch (config_dock_pos) {
-    case DockPos_Floating:
-        dock->x = config_dock_x;
-        dock->y = config_dock_y;
-        gravity = NorthWestGravity;
-        break;
-    case DockPos_TopLeft:
-        dock->x = 0;
-        dock->y = 0;
+    if (config_dock_floating) {
+        dock->area.x = config_dock_x;
+        dock->area.y = config_dock_y;
         gravity = NorthWestGravity;
-        break;
-    case DockPos_Top:
-        dock->x = screen_physical_size.width / 2;
-        dock->y = 0;
-        gravity = NorthGravity;
-        break;
-    case DockPos_TopRight:
-        dock->x = screen_physical_size.width;
-        dock->y = 0;
-        gravity = NorthEastGravity;
-        break;
-    case DockPos_Left:
-        dock->x = 0;
-        dock->y = screen_physical_size.height / 2;
-        gravity = WestGravity;
-        break;
-    case DockPos_Right:
-        dock->x = screen_physical_size.width;
-        dock->y = screen_physical_size.height / 2;
-        gravity = EastGravity;
-        break;
-    case DockPos_BottomLeft:
-        dock->x = 0;
-        dock->y = screen_physical_size.height;
-        gravity = SouthWestGravity;
-        break;
-    case DockPos_Bottom:
-        dock->x = screen_physical_size.width / 2;
-        dock->y = screen_physical_size.height;
-        gravity = SouthGravity;
-        break;
-    case DockPos_BottomRight:
-        dock->x = screen_physical_size.width;
-        dock->y = screen_physical_size.height;
-        gravity = SouthEastGravity;
-        break;
+    } else {
+        switch (config_dock_pos) {
+        case OB_DIRECTION_NORTHWEST:
+            dock->area.x = 0;
+            dock->area.y = 0;
+            gravity = NorthWestGravity;
+            break;
+        case OB_DIRECTION_NORTH:
+            dock->area.x = a->width / 2;
+            dock->area.y = 0;
+            gravity = NorthGravity;
+            break;
+        case OB_DIRECTION_NORTHEAST:
+            dock->area.x = a->width;
+            dock->area.y = 0;
+            gravity = NorthEastGravity;
+            break;
+        case OB_DIRECTION_WEST:
+            dock->area.x = 0;
+            dock->area.y = a->height / 2;
+            gravity = WestGravity;
+            break;
+        case OB_DIRECTION_EAST:
+            dock->area.x = a->width;
+            dock->area.y = a->height / 2;
+            gravity = EastGravity;
+            break;
+        case OB_DIRECTION_SOUTHWEST:
+            dock->area.x = 0;
+            dock->area.y = a->height;
+            gravity = SouthWestGravity;
+            break;
+        case OB_DIRECTION_SOUTH:
+            dock->area.x = a->width / 2;
+            dock->area.y = a->height;
+            gravity = SouthGravity;
+            break;
+        case OB_DIRECTION_SOUTHEAST:
+            dock->area.x = a->width;
+            dock->area.y = a->height;
+            gravity = SouthEastGravity;
+            break;
+        default:
+            g_assert_not_reached();
+        }
     }
 
     switch(gravity) {
     case NorthGravity:
     case CenterGravity:
     case SouthGravity:
-        dock->x -= dock->w / 2;
+        dock->area.x -= dock->area.width / 2;
         break;
     case NorthEastGravity:
     case EastGravity:
     case SouthEastGravity:
-        dock->x -= dock->w;
+        dock->area.x -= dock->area.width;
         break;
     }
     switch(gravity) {
     case WestGravity:
     case CenterGravity:
     case EastGravity:
-        dock->y -= dock->h / 2;
+        dock->area.y -= dock->area.height / 2;
         break;
     case SouthWestGravity:
     case SouthGravity:
     case SouthEastGravity:
-        dock->y -= dock->h;
+        dock->area.y -= dock->area.height;
         break;
     }
 
     if (config_dock_hide && dock->hidden) {
+        if (!config_dock_floating) {
+            switch (config_dock_pos) {
+            case OB_DIRECTION_NORTHWEST:
+                switch (config_dock_orient) {
+                case OB_ORIENTATION_HORZ:
+                    dock->area.y -= dock->area.height - ob_rr_theme->obwidth;
+                    break;
+                case OB_ORIENTATION_VERT:
+                    dock->area.x -= dock->area.width - ob_rr_theme->obwidth;
+                    break;
+                }
+                break;
+            case OB_DIRECTION_NORTH:
+                dock->area.y -= dock->area.height - ob_rr_theme->obwidth;
+                break;
+            case OB_DIRECTION_NORTHEAST:
+                switch (config_dock_orient) {
+                case OB_ORIENTATION_HORZ:
+                    dock->area.y -= dock->area.height - ob_rr_theme->obwidth;
+                    break;
+                case OB_ORIENTATION_VERT:
+                    dock->area.x += dock->area.width - ob_rr_theme->obwidth;
+                    break;
+                }
+                break;
+            case OB_DIRECTION_WEST:
+                dock->area.x -= dock->area.width - ob_rr_theme->obwidth;
+                break;
+            case OB_DIRECTION_EAST:
+                dock->area.x += dock->area.width - ob_rr_theme->obwidth;
+                break;
+            case OB_DIRECTION_SOUTHWEST:
+                switch (config_dock_orient) {
+                case OB_ORIENTATION_HORZ:
+                    dock->area.y += dock->area.height - ob_rr_theme->obwidth;
+                    break;
+                case OB_ORIENTATION_VERT:
+                    dock->area.x -= dock->area.width - ob_rr_theme->obwidth;
+                    break;
+                } break;
+            case OB_DIRECTION_SOUTH:
+                dock->area.y += dock->area.height - ob_rr_theme->obwidth;
+                break;
+            case OB_DIRECTION_SOUTHEAST:
+                switch (config_dock_orient) {
+                case OB_ORIENTATION_HORZ:
+                    dock->area.y += dock->area.height - ob_rr_theme->obwidth;
+                    break;
+                case OB_ORIENTATION_VERT:
+                    dock->area.x += dock->area.width - ob_rr_theme->obwidth;
+                    break;
+                }
+                break;
+            }    
+        }
+    }
+
+    if (!config_dock_floating && config_dock_hide) {
+        strw = ob_rr_theme->obwidth;
+        strh = ob_rr_theme->obwidth;
+    } else {
+        strw = dock->area.width;
+        strh = dock->area.height;
+    }
+
+    /* set the strut */
+    if (!dock->dock_apps) {
+        STRUT_PARTIAL_SET(dock_strut, 0, 0, 0, 0,
+                          0, 0, 0, 0, 0, 0, 0, 0);
+    } else if (config_dock_floating || config_dock_nostrut)
+    {
+        STRUT_PARTIAL_SET(dock_strut, 0, 0, 0, 0,
+                          0, 0, 0, 0, 0, 0, 0, 0);
+    } else {
         switch (config_dock_pos) {
-        case DockPos_Floating:
-            break;
-        case DockPos_TopLeft:
-            if (config_dock_horz)
-                dock->y -= dock->h - ob_rr_theme->bwidth;
-            else
-                dock->x -= dock->w - ob_rr_theme->bwidth;
+        case OB_DIRECTION_NORTHWEST:
+            switch (config_dock_orient) {
+            case OB_ORIENTATION_HORZ:
+                STRUT_PARTIAL_SET(dock_strut, 0, strh, 0, 0,
+                                  0, 0, dock->area.x, dock->area.x + dock->area.width - 1,
+                                  0, 0, 0, 0);
+                break;
+            case OB_ORIENTATION_VERT:
+                STRUT_PARTIAL_SET(dock_strut, strw, 0, 0, 0,
+                                  dock->area.y, dock->area.y + dock->area.height - 1,
+                                  0, 0, 0, 0, 0, 0);
+                break;
+            }
             break;
-        case DockPos_Top:
-            dock->y -= dock->h - ob_rr_theme->bwidth;
+        case OB_DIRECTION_NORTH:
+            STRUT_PARTIAL_SET(dock_strut, 0, strh, 0, 0,
+                              0, 0, dock->area.x, dock->area.x + dock->area.width - 1,
+                              0, 0, 0, 0);
             break;
-        case DockPos_TopRight:
-            if (config_dock_horz)
-                dock->y -= dock->h - ob_rr_theme->bwidth;
-            else
-                dock->x += dock->w - ob_rr_theme->bwidth;
+        case OB_DIRECTION_NORTHEAST:
+            switch (config_dock_orient) {
+            case OB_ORIENTATION_HORZ:
+                STRUT_PARTIAL_SET(dock_strut, 0, strh, 0, 0,
+                                  0, 0, dock->area.x, dock->area.x + dock->area.width -1,
+                                  0, 0, 0, 0);
+                break;
+            case OB_ORIENTATION_VERT:
+                STRUT_PARTIAL_SET(dock_strut, 0, 0, strw, 0,
+                                  0, 0, 0, 0,
+                                  dock->area.y, dock->area.y + dock->area.height - 1, 0, 0);
+                break;
+            }
             break;
-        case DockPos_Left:
-            dock->x -= dock->w - ob_rr_theme->bwidth;
+        case OB_DIRECTION_WEST:
+            STRUT_PARTIAL_SET(dock_strut, strw, 0, 0, 0,
+                              dock->area.y, dock->area.y + dock->area.height - 1,
+                              0, 0, 0, 0, 0, 0);
             break;
-        case DockPos_Right:
-            dock->x += dock->w - ob_rr_theme->bwidth;
+        case OB_DIRECTION_EAST:
+            STRUT_PARTIAL_SET(dock_strut, 0, 0, strw, 0,
+                              0, 0, 0, 0,
+                              dock->area.y, dock->area.y + dock->area.height - 1, 0, 0);
             break;
-        case DockPos_BottomLeft:
-            if (config_dock_horz)
-                dock->y += dock->h - ob_rr_theme->bwidth;
-            else
-                dock->x -= dock->w - ob_rr_theme->bwidth;
+        case OB_DIRECTION_SOUTHWEST:
+            switch (config_dock_orient) {
+            case OB_ORIENTATION_HORZ:
+                STRUT_PARTIAL_SET(dock_strut, 0, 0, 0, strh,
+                                  0, 0, 0, 0, 0, 0,
+                                  dock->area.x, dock->area.x + dock->area.width - 1);
+                break;
+            case OB_ORIENTATION_VERT:
+                STRUT_PARTIAL_SET(dock_strut, strw, 0, 0, 0,
+                                  dock->area.y, dock->area.y + dock->area.height - 1,
+                                  0, 0, 0, 0, 0, 0);
+                break;
+            }
             break;
-        case DockPos_Bottom:
-            dock->y += dock->h - ob_rr_theme->bwidth;
+        case OB_DIRECTION_SOUTH:
+            STRUT_PARTIAL_SET(dock_strut, 0, 0, 0, strh,
+                              0, 0, 0, 0, 0, 0,
+                              dock->area.x, dock->area.x + dock->area.width - 1);
             break;
-        case DockPos_BottomRight:
-            if (config_dock_horz)
-                dock->y += dock->h - ob_rr_theme->bwidth;
-            else
-                dock->x += dock->w - ob_rr_theme->bwidth;
+        case OB_DIRECTION_SOUTHEAST:
+            switch (config_dock_orient) {
+            case OB_ORIENTATION_HORZ:
+                STRUT_PARTIAL_SET(dock_strut, 0, 0, 0, strh,
+                                  0, 0, 0, 0, 0, 0,
+                                  dock->area.x, dock->area.x + dock->area.width - 1);
+                break;
+            case OB_ORIENTATION_VERT:
+                STRUT_PARTIAL_SET(dock_strut, 0, 0, strw, 0,
+                                  0, 0, 0, 0,
+                                  dock->area.y, dock->area.y + dock->area.height - 1, 0, 0);
+                break;
+            }
             break;
-        }    
-    }
-
-    /* set the strut */
-    switch (config_dock_pos) {
-    case DockPos_Floating:
-        STRUT_SET(dock_strut, 0, 0, 0, 0);
-        break;
-    case DockPos_TopLeft:
-        if (config_dock_horz)
-            STRUT_SET(dock_strut, 0, dock->h, 0, 0);
-        else
-            STRUT_SET(dock_strut, dock->w, 0, 0, 0);
-        break;
-    case DockPos_Top:
-        STRUT_SET(dock_strut, 0, dock->h, 0, 0);
-        break;
-    case DockPos_TopRight:
-        if (config_dock_horz)
-            STRUT_SET(dock_strut, 0, dock->h, 0, 0);
-        else
-            STRUT_SET(dock_strut, 0, 0, dock->w, 0);
-        break;
-    case DockPos_Left:
-        STRUT_SET(dock_strut, dock->w, 0, 0, 0);
-        break;
-    case DockPos_Right:
-        STRUT_SET(dock_strut, 0, 0, dock->w, 0);
-        break;
-    case DockPos_BottomLeft:
-        if (config_dock_horz)
-            STRUT_SET(dock_strut, 0, 0, 0, dock->h);
-        else
-            STRUT_SET(dock_strut, dock->w, 0, 0, 0);
-        break;
-    case DockPos_Bottom:
-        STRUT_SET(dock_strut, 0, 0, 0, dock->h);
-        break;
-    case DockPos_BottomRight:
-        if (config_dock_horz)
-            STRUT_SET(dock_strut, 0, 0, 0, dock->h);
-        else
-            STRUT_SET(dock_strut, 0, 0, dock->w, 0);
-        break;
+        }
     }
 
-    dock->w += minw;
-    dock->h += minh;
-
     /* not used for actually sizing shit */
-    dock->w -= ob_rr_theme->bwidth * 2;
-    dock->h -= ob_rr_theme->bwidth * 2;
+    dock->area.width -= ob_rr_theme->obwidth * 2;
+    dock->area.height -= ob_rr_theme->obwidth * 2;
+
+    if (dock->dock_apps) {
+        g_assert(dock->area.width > 0);
+        g_assert(dock->area.height > 0);
 
-    if (dock->w > 0 && dock->h > 0) {
         XMoveResizeWindow(ob_display, dock->frame,
-                          dock->x, dock->y, dock->w, dock->h);
+                          dock->area.x, dock->area.y, dock->area.width, dock->area.height);
 
-        RrPaint(dock->a_frame, dock->frame, dock->w, dock->h);
+        RrPaint(dock->a_frame, dock->frame, dock->area.width, dock->area.height);
         XMapWindow(ob_display, dock->frame);
     } else
         XUnmapWindow(ob_display, dock->frame);
 
     /* but they are useful outside of this function! */
-    dock->w += ob_rr_theme->bwidth * 2;
-    dock->h += ob_rr_theme->bwidth * 2;
+    dock->area.width += ob_rr_theme->obwidth * 2;
+    dock->area.height += ob_rr_theme->obwidth * 2;
+
+    screen_update_areas();
 
-    screen_update_struts();
+    g_free(a);
 }
 
-void dock_app_configure(DockApp *app, int w, int h)
+void dock_app_configure(ObDockApp *app, gint w, gint h)
 {
     app->w = w;
     app->h = h;
     dock_configure();
 }
 
-void dock_app_drag(DockApp *app, XMotionEvent *e)
+void dock_app_drag(ObDockApp *app, XMotionEvent *e)
 {
-    DockApp *over = NULL;
+    ObDockApp *over = NULL;
     GList *it;
-    int x, y;
+    gint x, y;
     gboolean after;
+    gboolean stop;
 
     x = e->x_root;
     y = e->y_root;
 
     /* are we on top of the dock? */
-    if (!(x >= dock->x &&
-          y >= dock->y &&
-          x < dock->x + dock->w &&
-          y < dock->y + dock->h))
+    if (!(x >= dock->area.x &&
+          y >= dock->area.y &&
+          x < dock->area.x + dock->area.width &&
+          y < dock->area.y + dock->area.height))
         return;
 
-    x -= dock->x;
-    y -= dock->y;
+    x -= dock->area.x;
+    y -= dock->area.y;
 
     /* which dock app are we on top of? */
-    for (it = dock->dock_apps; it; it = it->next) {
+    stop = FALSE;
+    for (it = dock->dock_apps; it; it = g_list_next(it)) {
         over = it->data;
-        if (config_dock_horz) {
+        switch (config_dock_orient) {
+        case OB_ORIENTATION_HORZ:
             if (x >= over->x && x < over->x + over->w)
-                break;
-        } else {
+                stop = TRUE;
+            break;
+        case OB_ORIENTATION_VERT:
             if (y >= over->y && y < over->y + over->h)
-                break;
+                stop = TRUE;
+            break;
         }
+        /* dont go to it->next! */
+        if (stop) break;
     }
     if (!it || app == over) return;
 
     x -= over->x;
     y -= over->y;
 
-    if (config_dock_horz)
+    switch (config_dock_orient) {
+    case OB_ORIENTATION_HORZ:
         after = (x > over->w / 2);
-    else
+        break;
+    case OB_ORIENTATION_VERT:
         after = (y > over->h / 2);
+        break;
+    default:
+        g_assert_not_reached();
+    }
 
     /* remove before doing the it->next! */
     dock->dock_apps = g_list_remove(dock->dock_apps, app);
@@ -430,35 +601,44 @@ void dock_app_drag(DockApp *app, XMotionEvent *e)
     dock_configure();
 }
 
-static void hide_timeout(void *n)
+static gboolean hide_timeout(gpointer data)
 {
-    /* dont repeat */
-    timer_stop(dock->hide_timer);
-    dock->hide_timer = NULL;
-
     /* hide */
     dock->hidden = TRUE;
     dock_configure();
+
+    return FALSE; /* don't repeat */
+}
+
+static gboolean show_timeout(gpointer data)
+{
+    /* hide */
+    dock->hidden = FALSE;
+    dock_configure();
+
+    return FALSE; /* don't repeat */
 }
 
 void dock_hide(gboolean hide)
 {
-    if (dock->hidden == hide || !config_dock_hide)
-        return;
     if (!hide) {
-        /* show */
-        dock->hidden = FALSE;
-        dock_configure();
-
-        /* if was hiding, stop it */
-        if (dock->hide_timer) {
-            timer_stop(dock->hide_timer);
-            dock->hide_timer = NULL;
+        if (dock->hidden && config_dock_hide) {
+            ob_main_loop_timeout_add(ob_main_loop, config_dock_show_delay,
+                                 show_timeout, NULL, g_direct_equal, NULL);
+        } else if (!dock->hidden && config_dock_hide) {
+            ob_main_loop_timeout_remove(ob_main_loop, hide_timeout);
         }
     } else {
-        g_assert(!dock->hide_timer);
-        dock->hide_timer = timer_start(config_dock_hide_timeout * 1000,
-                                       (TimeoutHandler)hide_timeout,
-                                       NULL);
+        if (!dock->hidden && config_dock_hide) {
+            ob_main_loop_timeout_add(ob_main_loop, config_dock_hide_delay,
+                                 hide_timeout, NULL, g_direct_equal, NULL);
+        } else if (dock->hidden && config_dock_hide) {
+            ob_main_loop_timeout_remove(ob_main_loop, show_timeout);
+        }
     }
 }
+
+void dock_get_area(Rect *a)
+{
+    RECT_SET(*a, dock->area.x, dock->area.y, dock->area.width, dock->area.height);
+}