More fixing for fade-outs.
[dana/xcompmgr.git] / xcompmgr.c
index a142cff..91ee807 100644 (file)
@@ -36,6 +36,7 @@
 #include <sys/time.h>
 #include <time.h>
 #include <unistd.h>
+#include <getopt.h>
 #include <X11/Xlib.h>
 #include <X11/Xutil.h>
 #include <X11/Xatom.h>
 
 #define CAN_DO_USABLE 0
 
+typedef enum {
+    WINTYPE_DESKTOP,
+    WINTYPE_DOCK,
+    WINTYPE_TOOLBAR,
+    WINTYPE_MENU,
+    WINTYPE_UTILITY,
+    WINTYPE_SPLASH,
+    WINTYPE_DIALOG,
+    WINTYPE_NORMAL,
+    WINTYPE_DROPDOWN_MENU,
+    WINTYPE_POPUP_MENU,
+    WINTYPE_TOOLTIP,
+    WINTYPE_NOTIFY,
+    WINTYPE_COMBO,
+    WINTYPE_DND,
+    NUM_WINTYPES
+} wintype;
+
 typedef struct _ignore {
     struct _ignore     *next;
     unsigned long      sequence;
@@ -79,8 +98,12 @@ typedef struct _win {
     int                        shadow_width;
     int                        shadow_height;
     unsigned int       opacity;
-    Atom                windowType;
+    wintype             windowType;
     unsigned long      damage_sequence;    /* sequence when damage was created */
+    Bool                destroyed;
+
+    Bool                need_configure;
+    XConfigureEvent     queue_configure;
 
     /* for drawing translucent windows */
     XserverRegion      borderClip;
@@ -98,9 +121,8 @@ typedef struct _fade {
     double             cur;
     double             finish;
     double             step;
-    void               (*callback) (Display *dpy, win *w, Bool gone);
+    void               (*callback) (Display *dpy, win *w);
     Display            *dpy;
-    Bool               gone;
 } fade;
 
 win             *list;
@@ -130,17 +152,14 @@ int               composite_opcode;
 /* find these once and be done with it */
 Atom           opacityAtom;
 Atom            winTypeAtom;
-Atom            winDesktopAtom;
-Atom            winDockAtom;
-Atom            winToolbarAtom;
-Atom            winMenuAtom;
-Atom            winUtilAtom;
-Atom            winSplashAtom;
-Atom            winDialogAtom;
-Atom            winNormalAtom;
+Atom            winType[NUM_WINTYPES];
+double          winTypeOpacity[NUM_WINTYPES];
+Bool            winTypeShadow[NUM_WINTYPES];
+Bool            winTypeFade[NUM_WINTYPES];
 
 /* opacity property name; sometime soon I'll write up an EWMH spec for it */
 #define OPACITY_PROP   "_NET_WM_WINDOW_OPACITY"
+#define REGISTER_PROP   "_NET_WM_CM_S"
 
 #define TRANSLUCENT    0xe0000000
 #define OPAQUE         0xffffffff
@@ -170,7 +189,7 @@ static void
 determine_mode(Display *dpy, win *w);
     
 static double
-get_opacity_percent(Display *dpy, win *w, double def);
+get_opacity_percent(Display *dpy, win *w);
 
 static XserverRegion
 win_extents (Display *dpy, win *w);
@@ -186,8 +205,6 @@ double  fade_in_step =  0.028;
 double  fade_out_step = 0.03;
 int    fade_delta =    10;
 int    fade_time =     0;
-Bool   fadeWindows = False;
-Bool    excludeDockShadows = False;
 Bool   fadeTrans = False;
 
 Bool   autoRedirect = False;
@@ -229,7 +246,7 @@ dequeue_fade (Display *dpy, fade *f)
        {
            *prev = f->next;
            if (f->callback)
-               (*f->callback) (dpy, f->w, f->gone);
+               (*f->callback) (dpy, f->w);
            free (f);
            break;
        }
@@ -254,8 +271,8 @@ enqueue_fade (Display *dpy, fade *f)
 
 static void
 set_fade (Display *dpy, win *w, double start, double finish, double step,
-         void (*callback) (Display *dpy, win *w, Bool gone),
-         Bool gone, Bool exec_callback, Bool override)
+         void (*callback) (Display *dpy, win *w),
+         Bool exec_callback, Bool override)
 {
     fade    *f;
 
@@ -274,7 +291,7 @@ set_fade (Display *dpy, win *w, double start, double finish, double step,
     {
        if (exec_callback)
            if (f->callback)
-               (*f->callback)(dpy, f->w, f->gone);
+               (*f->callback)(dpy, f->w);
     }
 
     if (finish < 0)
@@ -287,7 +304,6 @@ set_fade (Display *dpy, win *w, double start, double finish, double step,
     else if (f->cur > finish)
        f->step = -step;
     f->callback = callback;
-    f->gone = gone;
     w->opacity = f->cur * OPAQUE;
 #if 0
     printf ("set_fade start %g step %g\n", f->cur, f->step);
@@ -297,8 +313,19 @@ set_fade (Display *dpy, win *w, double start, double finish, double step,
     {
        XRenderFreePicture (dpy, w->shadow);
        w->shadow = None;
-       w->extents = win_extents (dpy, w);
+
+        if (w->extents != None)
+            XFixesDestroyRegion (dpy, w->extents);
+
+        /* rebuild the shadow */
+        w->extents = win_extents (dpy, w);
     }
+
+    /* fading windows need to be drawn, mark them as damaged.
+       when a window maps, if it tries to fade in but it already at the right
+       opacity (map/unmap/map fast) then it will never get drawn without this
+       until it repaints */
+    w->damaged = 1;
 }
 
 int
@@ -320,7 +347,7 @@ void
 run_fades (Display *dpy)
 {
     int            now = get_time_in_milliseconds();
-    fade    *f, *next;
+    fade    *next = fades;
     int            steps;
     Bool    need_dequeue;
 
@@ -330,8 +357,10 @@ run_fades (Display *dpy)
     if (fade_time - now > 0)
        return;
     steps = 1 + (now - fade_time) / fade_delta;
-    for (next = fades; f = next; )
+
+    while (next)
     {
+       fade *f = next;
        win *w = f->w;
        next = f->next;
        f->cur += f->step * steps;
@@ -365,6 +394,11 @@ run_fades (Display *dpy)
        {
            XRenderFreePicture (dpy, w->shadow);
            w->shadow = None;
+
+            if (w->extents != None)
+                XFixesDestroyRegion (dpy, w->extents);
+
+            /* rebuild the shadow */
            w->extents = win_extents(dpy, w);
        }
        /* Must do this last as it might destroy f->w in callbacks */
@@ -746,12 +780,12 @@ find_win (Display *dpy, Window id)
     win        *w;
 
     for (w = list; w; w = w->next)
-       if (w->id == id)
+       if (w->id == id && !w->destroyed)
            return w;
     return 0;
 }
 
-static char *backgroundProps[] = {
+static const char *backgroundProps[] = {
     "_XROOTPMAP_ID",
     "_XSETROOT_ID",
     0,
@@ -827,7 +861,7 @@ win_extents (Display *dpy, win *w)
     r.y = w->a.y;
     r.width = w->a.width + w->a.border_width * 2;
     r.height = w->a.height + w->a.border_width * 2;
-    if (compMode != CompSimple && !(w->windowType == winDockAtom && excludeDockShadows))
+    if (winTypeShadow[w->windowType])
     {
        if (compMode == CompServerShadows || w->mode != WINDOW_ARGB)
        {
@@ -956,8 +990,10 @@ paint_all (Display *dpy, XserverRegion region)
            Drawable                    draw = w->id;
            
 #if HAS_NAME_WINDOW_PIXMAP
-           if (hasNamePixmap && !w->pixmap)
-               w->pixmap = XCompositeNameWindowPixmap (dpy, w->id);
+           if (hasNamePixmap && !w->pixmap) {
+                set_ignore (dpy, NextRequest (dpy));
+                w->pixmap = XCompositeNameWindowPixmap (dpy, w->id);
+            }
            if (w->pixmap)
                draw = w->pixmap;
 #endif
@@ -1033,38 +1069,33 @@ paint_all (Display *dpy, XserverRegion region)
     for (w = t; w; w = w->prev_trans)
     {
        XFixesSetPictureClipRegion (dpy, rootBuffer, 0, 0, w->borderClip);
-       switch (compMode) {
-       case CompSimple:
-           break;
-       case CompServerShadows:
-           /* dont' bother drawing shadows on desktop windows */
-           if (w->windowType == winDesktopAtom)
-               break;
-           set_ignore (dpy, NextRequest (dpy));
-           if (w->opacity != OPAQUE && !w->shadowPict)
-               w->shadowPict = solid_picture (dpy, True,
-                                              (double) w->opacity / OPAQUE * 0.3,
-                                              0, 0, 0);
-           XRenderComposite (dpy, PictOpOver, 
-                             w->shadowPict ? w->shadowPict : transBlackPicture,
-                             w->picture, rootBuffer,
-                             0, 0, 0, 0,
-                             w->a.x + w->shadow_dx,
-                             w->a.y + w->shadow_dy,
-                             w->shadow_width, w->shadow_height);
-           break;
-       case CompClientShadows:
-           /* don't bother drawing shadows on desktop windows */
-           if (w->shadow && w->windowType != winDesktopAtom)
-           {
+        if (winTypeShadow[w->windowType]) {
+            switch (compMode) {
+            case CompSimple:
+                break;
+            case CompServerShadows:
+                set_ignore (dpy, NextRequest (dpy));
+                if (w->opacity != OPAQUE && !w->shadowPict)
+                    w->shadowPict = solid_picture (dpy, True,
+                                                   (double) w->opacity / OPAQUE * 0.3,
+                                                   0, 0, 0);
+                XRenderComposite (dpy, PictOpOver,
+                                  w->shadowPict ? w->shadowPict : transBlackPicture,
+                                  w->picture, rootBuffer,
+                                  0, 0, 0, 0,
+                                  w->a.x + w->shadow_dx,
+                                  w->a.y + w->shadow_dy,
+                                  w->shadow_width, w->shadow_height);
+                break;
+            case CompClientShadows:
                XRenderComposite (dpy, PictOpOver, blackPicture, w->shadow, rootBuffer,
                                  0, 0, 0, 0,
                                  w->a.x + w->shadow_dx,
                                  w->a.y + w->shadow_dy,
                                  w->shadow_width, w->shadow_height);
-           }
-           break;
-       }
+                break;
+            }
+        }
        if (w->opacity != OPAQUE && !w->alphaPict)
            w->alphaPict = solid_picture (dpy, False, 
                                          (double) w->opacity / OPAQUE, 0, 0, 0);
@@ -1163,10 +1194,118 @@ repair_win (Display *dpy, win *w)
     w->damaged = 1;
 }
 
+static const char*
+wintype_name(wintype type)
+{
+    const char *t;
+    switch (type) {
+    case WINTYPE_DESKTOP: t = "desktop"; break;
+    case WINTYPE_DOCK:    t = "dock"; break;
+    case WINTYPE_TOOLBAR: t = "toolbar"; break;
+    case WINTYPE_MENU:    t = "menu"; break;
+    case WINTYPE_UTILITY: t = "utility"; break;
+    case WINTYPE_SPLASH:  t = "slash"; break;
+    case WINTYPE_DIALOG:  t = "dialog"; break;
+    case WINTYPE_NORMAL:  t = "normal"; break;
+    case WINTYPE_DROPDOWN_MENU: t = "dropdown"; break;
+    case WINTYPE_POPUP_MENU: t = "popup"; break;
+    case WINTYPE_TOOLTIP: t = "tooltip"; break;
+    case WINTYPE_NOTIFY:  t = "notification"; break;
+    case WINTYPE_COMBO:   t = "combo"; break;
+    case WINTYPE_DND:     t = "dnd"; break;
+    default:              t = "unknown"; break;
+    }
+    return t;
+}
+
+static wintype
+get_wintype_prop(Display * dpy, Window w)
+{
+    Atom actual;
+    wintype ret;
+    int format;
+    unsigned long n, left, off;
+    unsigned char *data;
+
+    ret = (wintype)-1;
+    off = 0;
+
+    do {
+        set_ignore (dpy, NextRequest (dpy));
+        int result = XGetWindowProperty (dpy, w, winTypeAtom, off, 1L, False,
+                                         XA_ATOM, &actual, &format,
+                                         &n, &left, &data);
+
+        if (result != Success)
+            break;
+        if (data != None)
+        {
+            int i;
+
+            for (i = 0; i < NUM_WINTYPES; ++i) {
+                Atom a;
+                memcpy (&a, data, sizeof (Atom));
+                if (a == winType[i]) {
+                    /* known type */
+                    ret = i;
+                    break;
+                }
+            }
+
+            XFree ( (void *) data);
+        }
+
+        ++off;
+    } while (left >= 4 && ret == (wintype)-1);
+
+    return ret;
+}
+
+static wintype
+determine_wintype (Display *dpy, Window w, Window top)
+{
+    Window       root_return, parent_return;
+    Window      *children = NULL;
+    unsigned int nchildren, i;
+    wintype      type;
+
+    type = get_wintype_prop (dpy, w);
+    if (type != (wintype)-1)
+       return type;
+
+    set_ignore (dpy, NextRequest (dpy));
+    if (!XQueryTree (dpy, w, &root_return, &parent_return, &children,
+                           &nchildren))
+    {
+       /* XQueryTree failed. */
+       if (children)
+           XFree ((void *)children);
+       return (wintype)-1;
+    }
+
+    for (i = 0;i < nchildren;i++)
+    {
+       type = determine_wintype (dpy, children[i], top);
+       if (type != (wintype)-1)
+           return type;
+    }
+
+    if (children)
+       XFree ((void *)children);
+
+    if (w != top)
+        return (wintype)-1;
+    else
+        return WINTYPE_NORMAL;
+}
+
 static unsigned int
 get_opacity_prop (Display *dpy, win *w, unsigned int def);
 
 static void
+configure_win (Display *dpy, XConfigureEvent *ce);
+
+static void
 map_win (Display *dpy, Window id, unsigned long sequence, Bool fade)
 {
     win                *w = find_win (dpy, id);
@@ -1175,12 +1314,16 @@ map_win (Display *dpy, Window id, unsigned long sequence, Bool fade)
        return;
 
     w->a.map_state = IsViewable;
-    
-    /* This needs to be here or else we lose transparency messages */
-    XSelectInput (dpy, id, PropertyChangeMask);
 
-    /* This needs to be here since we don't get PropertyNotify when unmapped */
+    w->windowType = determine_wintype (dpy, w->id, w->id);
+#if 0
+    printf("window 0x%x type %s\n", w->id, wintype_name(w->windowType));
+#endif
+
+    /* select before reading the property so that no property changes are lost */
+    XSelectInput (dpy, id, PropertyChangeMask);
     w->opacity = get_opacity_prop (dpy, w, OPAQUE);
+
     determine_mode (dpy, w);
 
 #if CAN_DO_USABLE
@@ -1189,8 +1332,13 @@ map_win (Display *dpy, Window id, unsigned long sequence, Bool fade)
 #endif
     w->damaged = 0;
 
-    if (fade && fadeWindows)
-       set_fade (dpy, w, 0, get_opacity_percent (dpy, w, 1.0), fade_in_step, 0, False, True, True);
+    if (fade && winTypeFade[w->windowType])
+       set_fade (dpy, w, 0, get_opacity_percent (dpy, w), fade_in_step, 0, True, True);
+
+    /* if any configure events happened while the window was unmapped, then
+       configure the window to its correct place */
+    if (w->need_configure)
+        configure_win (dpy, &w->queue_configure);
 }
 
 static void
@@ -1221,10 +1369,6 @@ finish_unmap_win (Display *dpy, win *w)
        w->picture = None;
     }
 
-    /* don't care about properties anymore */
-    set_ignore (dpy, NextRequest (dpy));
-    XSelectInput(dpy, w->id, 0);
-
     if (w->borderSize)
     {
        set_ignore (dpy, NextRequest (dpy));
@@ -1247,7 +1391,7 @@ finish_unmap_win (Display *dpy, win *w)
 
 #if HAS_NAME_WINDOW_PIXMAP
 static void
-unmap_callback (Display *dpy, win *w, Bool gone)
+unmap_callback (Display *dpy, win *w)
 {
     finish_unmap_win (dpy, w);
 }
@@ -1260,9 +1404,14 @@ unmap_win (Display *dpy, Window id, Bool fade)
     if (!w)
        return;
     w->a.map_state = IsUnmapped;
+
+    /* don't care about properties anymore */
+    set_ignore (dpy, NextRequest (dpy));
+    XSelectInput(dpy, w->id, 0);
+
 #if HAS_NAME_WINDOW_PIXMAP
-    if (w->pixmap && fade && fadeWindows)
-       set_fade (dpy, w, w->opacity*1.0/OPAQUE, 0.0, fade_out_step, unmap_callback, False, False, True);
+    if (w->pixmap && fade && winTypeFade[w->windowType])
+       set_fade (dpy, w, w->opacity*1.0/OPAQUE, 0.0, fade_out_step, unmap_callback, False, True);
     else
 #endif
        finish_unmap_win (dpy, w);
@@ -1298,39 +1447,14 @@ get_opacity_prop(Display *dpy, win *w, unsigned int def)
    otherwise: the value
 */
 static double
-get_opacity_percent(Display *dpy, win *w, double def)
+get_opacity_percent(Display *dpy, win *w)
 {
+    double def = winTypeOpacity[w->windowType];
     unsigned int opacity = get_opacity_prop (dpy, w, (unsigned int)(OPAQUE*def));
 
     return opacity*1.0/OPAQUE;
 }
 
-/* determine mode for window all in one place.
-   Future might check for menu flag and other cool things
-*/
-
-static Atom
-get_wintype_prop(Display * dpy, Window w)
-{
-    Atom actual;
-    int format;
-    unsigned long n, left;
-
-    unsigned char *data;
-    int result = XGetWindowProperty (dpy, w, winTypeAtom, 0L, 1L, False,
-                                    XA_ATOM, &actual, &format,
-                                    &n, &left, &data);
-
-    if (result == Success && data != None)
-    {
-       Atom a;
-       memcpy (&a, data, sizeof (Atom));
-       XFree ( (void *) data);
-       return a;
-    }
-    return winNormalAtom;
-}
-
 static void
 determine_mode(Display *dpy, win *w)
 {
@@ -1381,40 +1505,6 @@ determine_mode(Display *dpy, win *w)
     }
 }
 
-static Atom
-determine_wintype (Display *dpy, Window w)
-{
-    Window       root_return, parent_return;
-    Window      *children = NULL;
-    unsigned int nchildren, i;
-    Atom         type;
-
-    type = get_wintype_prop (dpy, w);
-    if (type != winNormalAtom)
-       return type;
-
-    if (!XQueryTree (dpy, w, &root_return, &parent_return, &children,
-                           &nchildren))
-    {
-       /* XQueryTree failed. */
-       if (children)
-           XFree ((void *)children);
-       return winNormalAtom;
-    }
-
-    for (i = 0;i < nchildren;i++)
-    {
-       type = determine_wintype (dpy, children[i]);
-       if (type != winNormalAtom)
-           return type;
-    }
-
-    if (children)
-       XFree ((void *)children);
-
-    return winNormalAtom;
-}
-
 static void
 add_win (Display *dpy, Window id, Window prev)
 {
@@ -1426,7 +1516,7 @@ add_win (Display *dpy, Window id, Window prev)
     if (prev)
     {
        for (p = &list; *p; p = &(*p)->next)
-           if ((*p)->id == prev)
+           if ((*p)->id == prev && !(*p)->destroyed)
                break;
     }
     else
@@ -1454,6 +1544,7 @@ add_win (Display *dpy, Window id, Window prev)
     else
     {
        new->damage_sequence = NextRequest (dpy);
+        set_ignore (dpy, NextRequest (dpy));
        new->damage = XDamageCreate (dpy, id, XDamageReportNonEmpty);
     }
     new->alphaPict = None;
@@ -1466,12 +1557,12 @@ add_win (Display *dpy, Window id, Window prev)
     new->shadow_width = 0;
     new->shadow_height = 0;
     new->opacity = OPAQUE;
+    new->destroyed = False;
+    new->need_configure = False;
 
     new->borderClip = None;
     new->prev_trans = 0;
 
-    new->windowType = determine_wintype (dpy, new->id);
-    
     new->next = *p;
     *p = new;
     if (new->a.map_state == IsViewable)
@@ -1500,7 +1591,7 @@ restack_win (Display *dpy, win *w, Window new_above)
        /* rehook */
        for (prev = &list; *prev; prev = &(*prev)->next)
        {
-           if ((*prev)->id == new_above)
+           if ((*prev)->id == new_above && !(*prev)->destroyed)
                break;
        }
        w->next = *prev;
@@ -1528,49 +1619,62 @@ configure_win (Display *dpy, XConfigureEvent *ce)
        }
        return;
     }
+
+    if (w->a.map_state == IsUnmapped) {
+        /* save the configure event for when the window maps */
+        w->need_configure = True;
+        w->queue_configure = *ce;
+    }
+    else {
+        w->need_configure = False;
+
 #if CAN_DO_USABLE
-    if (w->usable)
+        if (w->usable)
 #endif
-    {
-       damage = XFixesCreateRegion (dpy, 0, 0);
-       if (w->extents != None) 
-           XFixesCopyRegion (dpy, damage, w->extents);
-    }
-    w->a.x = ce->x;
-    w->a.y = ce->y;
-    if (w->a.width != ce->width || w->a.height != ce->height)
-    {
+        {
+            damage = XFixesCreateRegion (dpy, 0, 0);
+            if (w->extents != None)
+                XFixesCopyRegion (dpy, damage, w->extents);
+        }
+
+        w->a.x = ce->x;
+        w->a.y = ce->y;
+        if (w->a.width != ce->width || w->a.height != ce->height)
+        {
 #if HAS_NAME_WINDOW_PIXMAP
-       if (w->pixmap)
-       {
-           XFreePixmap (dpy, w->pixmap);
-           w->pixmap = None;
-           if (w->picture)
-           {
-               XRenderFreePicture (dpy, w->picture);
-               w->picture = None;
-           }
-       }
+            if (w->pixmap)
+            {
+                XFreePixmap (dpy, w->pixmap);
+                w->pixmap = None;
+                if (w->picture)
+                {
+                    XRenderFreePicture (dpy, w->picture);
+                    w->picture = None;
+                }
+            }
 #endif
-       if (w->shadow)
-       {
-           XRenderFreePicture (dpy, w->shadow);
-           w->shadow = None;
-       }
+            if (w->shadow)
+            {
+                XRenderFreePicture (dpy, w->shadow);
+                w->shadow = None;
+            }
+        }
+        w->a.width = ce->width;
+        w->a.height = ce->height;
+        w->a.border_width = ce->border_width;
+        if (w->a.map_state != IsUnmapped && damage)
+        {
+            XserverRegion      extents = win_extents (dpy, w);
+            XFixesUnionRegion (dpy, damage, damage, extents);
+            XFixesDestroyRegion (dpy, extents);
+            add_damage (dpy, damage);
+        }
+
+        clipChanged = True;
     }
-    w->a.width = ce->width;
-    w->a.height = ce->height;
-    w->a.border_width = ce->border_width;
+
     w->a.override_redirect = ce->override_redirect;
     restack_win (dpy, w, ce->above);
-    if (damage)
-    {
-       XserverRegion   extents = win_extents (dpy, w);
-       XFixesUnionRegion (dpy, damage, damage, extents);
-       XFixesDestroyRegion (dpy, extents);
-       add_damage (dpy, damage);
-    }
-    clipChanged = True;
 }
 
 static void
@@ -1591,22 +1695,15 @@ circulate_win (Display *dpy, XCirculateEvent *ce)
 }
 
 static void
-finish_destroy_win (Display *dpy, Window id, Bool gone)
+finish_destroy_win (Display *dpy, Window id)
 {
     win        **prev, *w;
 
     for (prev = &list; (w = *prev); prev = &w->next)
-       if (w->id == id)
+       if (w->id == id && w->destroyed)
        {
-           if (gone)
-               finish_unmap_win (dpy, w);
+            finish_unmap_win (dpy, w);
            *prev = w->next;
-           if (w->picture)
-           {
-               set_ignore (dpy, NextRequest (dpy));
-               XRenderFreePicture (dpy, w->picture);
-               w->picture = None;
-           }
            if (w->alphaPict)
            {
                XRenderFreePicture (dpy, w->alphaPict);
@@ -1623,6 +1720,7 @@ finish_destroy_win (Display *dpy, Window id, Bool gone)
                XDamageDestroy (dpy, w->damage);
                w->damage = None;
            }
+
            cleanup_fade (dpy, w);
            free (w);
            break;
@@ -1631,23 +1729,27 @@ finish_destroy_win (Display *dpy, Window id, Bool gone)
 
 #if HAS_NAME_WINDOW_PIXMAP
 static void
-destroy_callback (Display *dpy, win *w, Bool gone)
+destroy_callback (Display *dpy, win *w)
 {
-    finish_destroy_win (dpy, w->id, gone);
+    finish_destroy_win (dpy, w->id);
 }
 #endif
 
 static void
-destroy_win (Display *dpy, Window id, Bool gone, Bool fade)
+destroy_win (Display *dpy, Window id, Bool fade)
 {
     win *w = find_win (dpy, id);
+
+    if (w) w->destroyed = True;
+
 #if HAS_NAME_WINDOW_PIXMAP
-    if (w && w->pixmap && fade && fadeWindows)
-       set_fade (dpy, w, w->opacity*1.0/OPAQUE, 0.0, fade_out_step, destroy_callback, gone, False, True);
+    if (w && w->pixmap && fade && winTypeFade[w->windowType])
+       set_fade (dpy, w, w->opacity*1.0/OPAQUE, 0.0, fade_out_step,
+                  destroy_callback, False, True);
     else
 #endif
     {
-       finish_destroy_win (dpy, id, gone);
+       finish_destroy_win (dpy, id);
     }
 }
 
@@ -1719,8 +1821,8 @@ damage_win (Display *dpy, XDamageNotifyEvent *de)
            w->a.height <= w->damage_bounds.y + w->damage_bounds.height)
        {
            clipChanged = True;
-           if (fadeWindows)
-               set_fade (dpy, w, 0, get_opacity_percent (dpy, w, 1.0), fade_in_step, 0, False, True, True);
+           if (winTypeFade[w->windowType])
+               set_fade (dpy, w, 0, get_opacity_percent (dpy, w), fade_in_step, 0, True, True);
            w->usable = True;
        }
     }
@@ -1733,7 +1835,7 @@ static int
 error (Display *dpy, XErrorEvent *ev)
 {
     int            o;
-    char    *name = 0;
+    const char    *name = 0;
     
     if (should_ignore (dpy, ev->serial))
        return 0;
@@ -1848,6 +1950,7 @@ usage (char *program)
     fprintf (stderr, "   -I fade-in-step\n      Specifies the opacity change between steps while fading in. (default 0.028)\n");
     fprintf (stderr, "   -O fade-out-step\n      Specifies the opacity change between steps while fading out. (default 0.03)\n");
     fprintf (stderr, "   -D fade-delta-time\n      Specifies the time between steps in a fade in milliseconds. (default 10)\n");
+    fprintf (stderr, "   -m opacity\n      Specifies the opacity for menus. (default 1.0)\n");
     fprintf (stderr, "   -a\n      Use automatic server-side compositing. Faster, but no special effects.\n");
     fprintf (stderr, "   -c\n      Draw client-side shadows with fuzzy edges.\n");
     fprintf (stderr, "   -C\n      Avoid drawing shadows on dock/panel windows.\n");
@@ -1860,10 +1963,14 @@ usage (char *program)
 }
 
 static void
-register_cm (void)
+register_cm (int scr)
 {
     Window w;
     Atom a;
+    char *buf;
+    int len, s;
+
+    if (scr < 0) return;
 
     w = XCreateSimpleWindow (dpy, RootWindow (dpy, 0), 0, 0, 1, 1, 0, None,
                             None);
@@ -1871,8 +1978,17 @@ register_cm (void)
     Xutf8SetWMProperties (dpy, w, "xcompmgr", "xcompmgr", NULL, 0, NULL, NULL,
                          NULL);
 
-    /* FIXME: Don't hard code the screen number */
-    a = XInternAtom (dpy, "_NET_WM_CM_S0", False);
+    len = strlen(REGISTER_PROP) + 2;
+    s = scr;
+    while (s >= 10) {
+        ++len;
+        s /= 10;
+    }
+    buf = malloc(len);
+    snprintf(buf, len, REGISTER_PROP"%d", scr);
+
+    a = XInternAtom (dpy, buf, False);
+    free(buf);
 
     XSetSelectionOwner (dpy, a, w, 0);
 }
@@ -1894,8 +2010,18 @@ main (int argc, char **argv)
     int                    composite_major, composite_minor;
     char           *display = 0;
     int                    o;
+    Bool            noDockShadow = False;
+
+    for (i = 0; i < NUM_WINTYPES; ++i) {
+        winTypeFade[i] = False;
+        winTypeShadow[i] = False;
+        winTypeOpacity[i] = 1.0;
+    }
+
+    /* don't bother to draw a shadow for the desktop */
+    winTypeShadow[WINTYPE_DESKTOP] = False;
 
-    while ((o = getopt (argc, argv, "D:I:O:d:r:o:l:t:scnfFCaS")) != -1)
+    while ((o = getopt (argc, argv, "D:I:O:d:r:o:m:l:t:scnfFCaS")) != -1)
     {
        switch (o) {
        case 'd':
@@ -1918,18 +2044,29 @@ main (int argc, char **argv)
            break;
        case 's':
            compMode = CompServerShadows;
+            for (i = 0; i < NUM_WINTYPES; ++i)
+                winTypeShadow[i] = True;
            break;
        case 'c':
            compMode = CompClientShadows;
+            for (i = 0; i < NUM_WINTYPES; ++i)
+                winTypeShadow[i] = True;
            break;
        case 'C':
-           excludeDockShadows = True;
+            noDockShadow = True;
            break;
        case 'n':
            compMode = CompSimple;
+            for (i = 0; i < NUM_WINTYPES; ++i)
+                winTypeShadow[i] = False;
+           break;
+       case 'm':
+            winTypeOpacity[WINTYPE_DROPDOWN_MENU] = atof (optarg);
+            winTypeOpacity[WINTYPE_POPUP_MENU] = atof (optarg);
            break;
        case 'f':
-           fadeWindows = True;
+            for (i = 0; i < NUM_WINTYPES; ++i)
+                winTypeFade[i] = True;
            break;
        case 'F':
            fadeTrans = True;
@@ -1957,6 +2094,9 @@ main (int argc, char **argv)
            break;
        }
     }
+
+    if (noDockShadow)
+        winTypeShadow[WINTYPE_DOCK] = False;
     
     dpy = XOpenDisplay (display);
     if (!dpy)
@@ -1998,19 +2138,25 @@ main (int argc, char **argv)
        exit (1);
     }
 
-    register_cm();
+    register_cm(scr);
 
     /* get atoms */
     opacityAtom = XInternAtom (dpy, OPACITY_PROP, False);
     winTypeAtom = XInternAtom (dpy, "_NET_WM_WINDOW_TYPE", False);
-    winDesktopAtom = XInternAtom (dpy, "_NET_WM_WINDOW_TYPE_DESKTOP", False);
-    winDockAtom = XInternAtom (dpy, "_NET_WM_WINDOW_TYPE_DOCK", False);
-    winToolbarAtom = XInternAtom (dpy, "_NET_WM_WINDOW_TYPE_TOOLBAR", False);
-    winMenuAtom = XInternAtom (dpy, "_NET_WM_WINDOW_TYPE_MENU", False);
-    winUtilAtom = XInternAtom (dpy, "_NET_WM_WINDOW_TYPE_UTILITY", False);
-    winSplashAtom = XInternAtom (dpy, "_NET_WM_WINDOW_TYPE_SPLASH", False);
-    winDialogAtom = XInternAtom (dpy, "_NET_WM_WINDOW_TYPE_DIALOG", False);
-    winNormalAtom = XInternAtom (dpy, "_NET_WM_WINDOW_TYPE_NORMAL", False);
+    winType[WINTYPE_DESKTOP] = XInternAtom (dpy, "_NET_WM_WINDOW_TYPE_DESKTOP", False);
+    winType[WINTYPE_DOCK] = XInternAtom (dpy, "_NET_WM_WINDOW_TYPE_DOCK", False);
+    winType[WINTYPE_TOOLBAR] = XInternAtom (dpy, "_NET_WM_WINDOW_TYPE_TOOLBAR", False);
+    winType[WINTYPE_MENU] = XInternAtom (dpy, "_NET_WM_WINDOW_TYPE_MENU", False);
+    winType[WINTYPE_UTILITY] = XInternAtom (dpy, "_NET_WM_WINDOW_TYPE_UTILITY", False);
+    winType[WINTYPE_SPLASH] = XInternAtom (dpy, "_NET_WM_WINDOW_TYPE_SPLASH", False);
+    winType[WINTYPE_DIALOG] = XInternAtom (dpy, "_NET_WM_WINDOW_TYPE_DIALOG", False);
+    winType[WINTYPE_NORMAL] = XInternAtom (dpy, "_NET_WM_WINDOW_TYPE_NORMAL", False);
+    winType[WINTYPE_DROPDOWN_MENU] = XInternAtom (dpy, "_NET_WM_WINDOW_TYPE_DROPDOWN_MENU", False);
+    winType[WINTYPE_POPUP_MENU] = XInternAtom (dpy, "_NET_WM_WINDOW_TYPE_POPUP_MENU", False);
+    winType[WINTYPE_TOOLTIP] = XInternAtom (dpy, "_NET_WM_WINDOW_TYPE_TOOLTIP", False);
+    winType[WINTYPE_NOTIFY] = XInternAtom (dpy, "_NET_WM_WINDOW_TYPE_NOTIFICATION", False);
+    winType[WINTYPE_COMBO] = XInternAtom (dpy, "_NET_WM_WINDOW_TYPE_COMBO", False);
+    winType[WINTYPE_DND] = XInternAtom (dpy, "_NET_WM_WINDOW_TYPE_DND", False);
 
     pa.subwindow_mode = IncludeInferiors;
 
@@ -2084,7 +2230,7 @@ main (int argc, char **argv)
                configure_win (dpy, &ev.xconfigure);
                break;
            case DestroyNotify:
-               destroy_win (dpy, ev.xdestroywindow.window, True, True);
+               destroy_win (dpy, ev.xdestroywindow.window, True);
                break;
            case MapNotify:
                map_win (dpy, ev.xmap.window, ev.xmap.serial, True);
@@ -2096,7 +2242,7 @@ main (int argc, char **argv)
                if (ev.xreparent.parent == root)
                    add_win (dpy, ev.xreparent.window, 0);
                else
-                   destroy_win (dpy, ev.xreparent.window, False, True);
+                   destroy_win (dpy, ev.xreparent.window, True);
                break;
            case CirculateNotify:
                circulate_win (dpy, &ev.xcirculate);
@@ -2154,16 +2300,21 @@ main (int argc, char **argv)
                    if (w)
                    {
                        if (fadeTrans)
-                           set_fade (dpy, w, w->opacity*1.0/OPAQUE, get_opacity_percent (dpy, w, 1.0),
-                                     fade_out_step, 0, False, True, False);
+                           set_fade (dpy, w, w->opacity*1.0/OPAQUE, get_opacity_percent (dpy, w),
+                                     fade_out_step, 0, True, False);
                        else
                        {
-                       w->opacity = get_opacity_prop(dpy, w, OPAQUE);
-                       determine_mode(dpy, w);
+                            w->opacity = get_opacity_prop(dpy, w, OPAQUE);
+                            determine_mode(dpy, w);
                            if (w->shadow)
                            {
                                XRenderFreePicture (dpy, w->shadow);
                                w->shadow = None;
+
+                                if (w->extents != None)
+                                    XFixesDestroyRegion (dpy, w->extents);
+
+                                /* rebuild the shadow */
                                w->extents = win_extents (dpy, w);
                            }
                        }