Some versions of X, have the Shape extension, but apparently not "ShapeInput" (Fixes bug #4662)

the sawfish window manager has ifdefs for this sort of situation.
I followed suit, and #ifdef'd it, and it now works for me.
patch attached.

Slight changes to the patch from danakj@orodu.net for readability

make the default drag threshold 1.  it is smooth and feels and looks nice.

(also macos has it and i liked it.)

XML_PARSE_XINCLUDE flag seems to do nothing. Use xmlXIncludeProcessFlags() instead

if no launch time is provided for an application, make one up.

if the window is related to other existing windows
  and one of those windows was the last used
    then we will give it a launch time equal to the last user time,
    which will end up giving the window focus probably.
  else
    the window is related to other windows, but you are not working in them?
    seems suspicious, so we will give it a launch time of NOW - STEAL_INTERVAL,
    so it will be given focus only if we didn't use something else during the
    steal interval.
else
  the window is all on its own, so we can't judge it.  give it a launch time
  equal to the last user time, so it will probably take focus.

this way running things from a terminal will give them focus, but popups
without a launch time shouldn't steal focus so easily.

fix segfault for placing windows without a group

add primary monitor stuff to rc.xsd

Fix small leak in If action option parsing

Big changes to placement across multiple monitors.

Add a Primary option for which monitor to place new windows on.  Make "Active" the default instead of "Any", which is just totally crazy.

When a window is being placed in the FOREGROUND, use a monitor chosen in
  the following order:
  1. same monitor as parent
  2. primary monitor if placement=PRIMARY
     active monitor if placement=ACTIVE
     pointer monitor if placement=MOUSE
  3. primary monitor
  4. other monitors where the window has group members on the same desktop
  5. other monitors where the window has group members on other desktops
  6. other monitors

When a window is being placed in the BACKGROUND, use a monitor chosen in the
  following order:
  1. same monitor as parent
  2. other monitors where the window has group members on the same desktop
   2a. primary monitor in this set
   2b. other monitors in this set
  3. other monitors where the window has group members on other desktops
   3a. primary monitor in this set
   3b. other monitors in this set
  4. other monitors
   4a. primary monitor in this set
   4b. other monitors in this set

Decide to focus the new window before placing it, so we know if it will be
  placed in the foreground or background.

Always choose a single monitor, then place on it, rather than possibly moving
  to a "backup" monitor.  Unpredictable monitor placement is horrible.

add function screen_compare_desktops()

tells if two windows' current desktops are considered logically on the same
  desktop (taking "all desktops" into account)
if a window is on "all desktops" it is considered to be on the current desktop
  only - windows can only be in one place at a time.

don't steal focus if someone used a window within 1 second ago

(instead of 0.5 seconds)

Fixing commit 066a15d7aa64eaa0.  Don't ever use CurrentTime as event_curtime.

Force event_curtime to be some valid timestamp.

Allow XIncludes in xml documents we read.

Add the XInclude namespace in the default config file (named "xi")

use top_builddir, not builddir variable in Makefile.  seems builddir doesn't always exist/anymore

add some more warning flags for debug builds

-Wno-write-strings so that "foo" is treated as a const char* not a char*.
-Wextra for more fun

Fix icons.c test for 64-bit platforms

update the last user interaction timestamp during a move/resize of the focused window

update the user-interaction timestamp when running a (non-interactive) action on the focused window
update it at the end of running actions instead of multiple times (once for
 each action)

don't steal focus from a window if it was used very recently, when someone uses _NET_ACTIVE request.

mem leak in config.c

Big rework of image.c and the image cache system.

Added a lot of comments, simplified call graphs.
Added full (not second-class) support for images coming from named sources (files, icon themes).
RrImage holds an RrImageSet.  RrImageSet holds a bunch of RrImagePic, which are different sizes of a logical image.
RrImageSet objects can be merged if it is discovered they (will) share an RrImagePic.  The RrImage objects are updated to use the new merged RrImageSet.

Use >= instead of > when comparing timestamps, or we will ignore some events

add some tests for obt pieces (binary search, .desktop parsking, linkbase, and inotify watch)

oops, crash if searching for something smaller than * in the array

less redundant calls to screen_update_areas() and client_move_onscreen()

Ignore MotionNotify on menus, it reset the submenudelay timer

It also caused the slightest cursor movement to activate the entry under
the cursor when navigating via the keyboard.

Use g_list_find instead of weird bouncing loops

Parse multiple keys in one keybind

allow the res_class in WM_CLASS to mark a window as a dock app, if it is set to the string "DockApp"

Don't hog the user during the hideDelay

Handle STRING and COMPOUND_TEXT type text properties.

STRING should be latin1 text (plus TAB and LF)
COMPOUND_TEXT should be encoded in the current locale.

Revert "add an inotify watcher for directories (doesnt work without inotify yet)"

This reverts commit c36b89ba12eae18d3011c8516906c21e9abb89dc.

fix make distcheck

glib can't handle -Wcast-qual

add wmhints test that toggles motif decor hints on a mapped window

Fix a double free when you have a mousebind without a button attribute

Introduced in f307a3feabedd9bcadeaafd0fa8e1b1a60736eb2

Fix a copypasto that broke interactive keybinds with meta and hyper keys

Revert "add XFlush to g_timeout callbacks"

This reverts commit aa3685d16af4a565eecdc39047ee8d140ef5cd99.

Mika's commit using a GSource to handle X Events fixes this problem, so this
commit is not needed

Fix 3.4 compatibility for SendToDesktop

Correct typo occured -> occurred in function name

Add signal handling with the GMainLoop

Provided through a very simplistic interface in obt, found in the
 obt/signal.[ch] files

add stackabove test, that lets you move a client window just above a specified window in the stacking order

80 cols

Continue parsing contexts when an unsupported one is encountered

Make the dock a context and add actions LowerDock and RaiseDock

Move usage of iconh into the branch that assigns it

settings can't be NULL here, or we'd have already crashed anyway

Use a gsource instead of gio channels for watching x events

Fixes not getting some focusin events sometimes.

obt returns libxml2 structures directly, so anyone linking against it better also link to libxml2

add XFlush to g_timeout callbacks

GMainLoop doesn't cause an flush to occur after handling a timeout.

Fix the EXTENTS reported on window frames.

Change the order things are done on map, so we apply startup state without
  any states set first, thus getting all the functions possible for the window
  type etc.  then change the order states are applied, as some remove the
  ability to apply others (ie fullscreen comes last).
Add an oldsize to ObFrame that remembers the size of the frame last reported to
  the world through the EXTENTS property.  If you frame_adjust_area(FAKE) then
  frame_adjust_area(NONFAKE), the EXTENTS would not be updated since the
  oldsize wasn't remembered across fake updates.
Make the extentsrequest test ask about windows with states set also.

keep stacking_list_tail a pointer to the tail of the stacking list

allow a user to set "undecorated" on a window without decorations

if not based on window type, the app may decide to have decorations again
sometime.  this was not always legal but chromium has made it so.

add an inotify watcher for directories (doesnt work without inotify yet)

Use GMainLoop instead of ObtMainLoop

save the list of mime types listed as supported by an application

parse app Categories better.  parse a list of strings into a gchar**

parse Categories tag and save them as an array of GQuarks

improved .desktop parsing.

properly (and quickly) check for existence of required keys
figure out what an app can open from its exec key
validate the %fields in an app's exec key

free the user's group ids

free the path dirs

parse Path, Terminal, and StartupNotify

parse OnlyShowIn/NotShowIn

allow new windows on other desktops to get focused if specified in a per-app rule

you can not client_active() with user=false and desktop=true and activate things on other desktops, by the general focus steal restrictions, if you wanted to do that sorta thing

Allow sending windows on other desktops to the current desktop.

Add some more conditions to the If action

It can now match on window titles, virtual desktop (number, "current",
"other"),
demands_attention, and undecoratedness.

don't need a force_reply when changing the decor for a window

it will send a configurenotify if the client's rootwindow position changed

client_update_protocols() doesnt change anything that would affect a client's decor or functions

allows a window to unmax by resizing.

makes "clever" use of the client's pre_max_area values to allow this to happen
without flashing, and preserving them in case the user cancels the resize
after it has become unmaximized.

set the frame's initial size to something invalid so the extents hint will be
set for sure the first time.

set the interactive action to NULL before doing its cleanup phase

if the cleanup ends up cancelling the "current interactive action", it would
cause the end function to be called again, which sucks.  for instance if the
cleanup function uses the Focus action. this prevents the unwanted loop
into the interactive action cleanup.

fix focus when starting up, especially when replacing another instance of openbox

adds a function event_reset_time() that forces event_time() to look for a new (future) timestamp

remove a debug print in the keyboard obt stuff

avoid new gcc warning (casting ** to const** is not valid for good reason)

merge some contexts together in the default rc.xml mouse bindings

allow multiple contexts separated by space in a mouse binding

example: context="Top Left Right Bottom"

turn off focuslast and on undermouse in mouse focus config example

Remove some unneeded stdlib.h includes

Add support for using relative expressions in move and resize actions

Have MoveResizeTo use config_parse_gravity_coord instead of duplicating it locally
Allow MoveResizeTo positions and sizes and per app positions to be relative to screen size
Rename to config_parse_relative_number so it can be used for sizes too
Add relative numbers to width/height in MoveResizeTo
Add relative numbers to MoveRelative
Add relative numbers to ResizeRelative, these are for the client size, not screen size

new enum values need to be handled in switches

(the new enum values were added to count the number of entries in the enums)

fix typo in enum

add _NUM_TYPES to obrender enums, to allow enumeration of the enum values

show empty properties in obxprop

parse number values write in a hex string in obxprop

advance while parsing a hex input in obxprop

add cleanup functions to the client list menus that removes everything from them when they are not being shown (saves iterating thru the list uselessly when a client is unmanaged)

add cleanup function for menus that fires when the menu is no longer visible

don't try go to selected submenu when there is nothing selected

use clicks in prompts

add a test to set a mouse cursor on an InputOnly subwindow

fix XSYNCronization of resizes (especially with two monitors).

1) a ConfigureNotify must always follow a Sync notification
2) determine the final size of the window with its position else struts are not applied properly for (partly) maximized windows
3) set the sync counter to a value when managing a new window

make sure clients stay on screen when the root window changes size, and maximized/fullscreen windows must be on some monitor or it doesn't make much sense (and asserts)

Remove two unused variables

Remains from 64adc0eeba598cb1469b2140777fba30e3053f0e

found memory corruption in the obt xqueue and paths

make an event queue for X events.  the queue's min size is 16 XEvents (~3k)

don't put reparent events back on the stack.  the window is no longer being managed anyways.  and it messes up the order of events in the queue

putting events on the event queue does not send them back to the server.  it was just going to get ignored the next time around cuz the window wouldnt be in our list of clients

don't use XPutBackEvent and mess up timestamp order in the event queue

add asserts to make sure we don't add things to stacking list that are not managed

don't change stacking stuff on windows when moving them during the unmanage phase.

event_curtime is replaced (publicly) by event_time()

This function never returns CurrentTime, which is nice, cuz using CurrentTime for XSetFocus always sucks.

If the current XEvent did not have a timestamp, then event_time() will find one.  It finds the first timestamp available in the X event queue, meaning the earliest timestamp >= the current (nontimestamped) event. All future events should have a timestamp >= event_time(), so using this in XSetFocus() should not mess up any future calls we make to it.

This change seems to work well, as it appears to fix bug #3648.