Merge branches 'wip/edges', 'mikabox/crap', 'mikabox/lock', 'mikabox/maybe' and 'mikabox/personal' into wip/mikabox

Conflicts:
obt/prop.c
openbox/client_list_combined_menu.c
openbox/event.c

Allow specifying several mousebinds in one go (hack)

Doesn't handle trivial stuff like multiple spaces, it's 3:24am so no guarantees otherwise either.

Start on the second entry...

Only warp on menu close if it was also the top level menu.

This might be a little bit better.

At least it consistently doesn't run the openbox action.

"Fix" releasing the mouse button over a window.

Previously if you click-drag-released the left button on the root window to a client window,
Openbox would ignore the release because we don't own the client window. Now check if we
are in a button press or not before ignoring it. The bug was we would initiate a move when
the cursor was moved over a titlebar even when the button was unpressed because the variable
didn't get cleared properly.

This fix is very ugly.

[0]. is confusing when it's not an array

Reinit edges stuff on screen resize

edges should hook into number of screens changing, but for now, don't crash at reconfigure/shutdown if they did

Handle edges in window.c

Allow clients to place themselves up to 99% offscreen.

Really what I want is keep the 10% but let them stay if they were already there. Most noticably steam jumps back to the 10% limit whenever you click anywhere because it stupidly moves itself to where it already is when you click anywhere inside it.

edges: do something clever with xinerama

This is useful now that X actually constrains the cursor to visible
areas of the screen, so you have a chance of hitting edges and possibly
corners too. More cleverness is possible, should not map edges where
there is a screen on both sides probably.

dunno why this was lying around as an uncommitted change..

WIP version 3 of edges thinger

This is lets you bind actions to clicking on screen edges/corners.
Eventually it should also let you bind hovering or entering/leaving and
whatnot.

Stop moving stuff onscreen when the screen size changes

Listen to key press events on the support window

This allows us to make binds that are only active when no clients are
focused with the grab=no option. However, this will mean that these
binds will also be launched if the keyboard is otherwise grabbed, for
example during alt-tab or when showing a menu.

Do focus menus with the mouse.

Crazy focus model.

I can drag on buttons.

Save screen_last_desktop to the root prop _OB_LAST_DESKTOP.

Resist on skip_taskbar windows too

Make the menu pop up centered below the mouse pointer.

also move the menu one pixel down so you can click and the menu doesn't stick.

Add a _OB_FOCUS client message for obpanel to use.

Disable the evil calling of a kde program on startup

Various style related changes.

This includes:
  Not forcing an even label_height
  Changing the cursor to BUSY (from BUSYPOINTER)
  Making the label area in the dirfocus popup different
  No menu item padding
  Nice blue colors for the focus cycle indicator
  Not making shadows increase text boundaries

Add a window icon that is more transparent to stand out less.

Put the client-list-menu in the client menu, just for fun. Possibly useful when there are a lot of maximized windows or the desktop is otherwise unreachable and you want to access an app without using the keyboard.

Change the client list menus back to separating iconified windows with a line instead of bracketing them.

Stop focus from moving if scroll lock is enabled. Useful when some program decides to pop up a window every 2 seconds while you're chatting away on irc.

Make the default window icon randomly tinted.

handle locked windows in client list menus also. this is a really stupid way of doing it.

Add "locked" to if action

Disallow some more stuff on locked windows

Don't allow deiconifying locked windows via the focus action.

Make * not crash when something isn't focused.

Set a prop on locked windows, so panels can see it, and so it persists over restarts.

Lock some stuff in event.c too.

Better locking for client menu.

Disable locking for some actions where it didn't make sense.

Add Lock action.

Allows locking windows so you can't move/resize/close/etc them by mistake.

It's probably better to check in client.c than the actions, just
testing if I want it at all for now.

Add FocusFallback action.

Make Unfocus do what the name says.

New SendKeyEvent and SetKeyTarget actions.

These let you define SendKeyEvent actions that send a specified key to a target
 window. The target window can be changed at runtime with the SetKeyTarget action.

Don't bring windows when uniconifying.

Add "de" option to Iconify action that makes it deiconify windows (useful while alt-tabbing).

Add RaiseTemp action.

This has the same effect as giving the <raise>yes</raise> option to next/prevwindow
but makes it optional and keybindable, so if W-Tab is nextwindow, you can have W-q
as raisetemp.

This commit is mostly a reminder to refactor the desktop popup creation, but it gives it the menu title font

Allow to use a pango font description string.

Add emulatexinerama option in desktops section that splits the screen in two side-by-side areas.

Warp to the top of the combined menu and back.

Put current desktop on top of combined client list menu.

This probably needs an option.

Center text in pager popup and make it at least as wide as it is high.

Show popup even earlier.

But ideally, the window should have moved/resized by now, it doesn't until the next pixel...

Add grab option to <keybind> to not grab the keycombo.

This is for example to let you bind stuff to the Alt key while alt-tabbing,
but you don't want those keys to be grabbed in normal usage.

Don't update hints when the title is the same.

What I really wanted was to not redraw the titlebar but that seems much harder.

Various uninteresting changes to the test programs.

Allow selecting the random placement strategy. The hunk for place.c that actually does this seems to have been lost somewhere on the way.

Commented version of color cycling core cursor. I think it leaks the color specs.

Pass the GravityPoint as const* instead of by value

Let the menu placement code know if the position was user specified

This way, it can avoid running the code that moves it to a good position
relative to a mouse that isn't there

Move common gravity application to screen.c

Resolve gravity coords in menuframe so it can use the widths

Add <position> tag to ShowMenu action

Fix negative relative coordinate placement

double space

Fix thinko in if.c, *o in Stop can never be pointing at the right place.

Use a file static boolean instead.

Fix typo in variable name

Avoid initializing a VLA of 0 length

Declare variables at start of blocks in place.c

Break loop if is_true was set to false in If

Don't segfault when there's no target in If

Some fixes in french translation

Cache results of offset/shift functions so we don't do 6 function calls per pixel when reducing color depth

Fix gcc warnings in screen.c

Added missing break statement (Bug 6043)

Add a strict option to the ToggleShowDesktop action

When the strict option is used, normal windows are not able to show themselves
while showing the desktop.

Portuguese update for openbox

Allow FillToEdge to grow when all its edges are blocked.

This change points out how useless the return value from client_find_resize_directional()
was, so it also removes that.

When all edges are blocked, the FillToEdge action will try to grow all four edges agin
without blocking any of them on their current edge. This more closely matches the
behaviour of the GrowToEdge action.

Clean up the FillToEdge action implementation

This extend the client_find_resize_directional() method to support
two growing modes, and return a bool for whether it was able to grow/shrink.

The client_find_resize_directional() method now takes an enum instead
of a bool, with two growing modes. The old mode which always tries to
grow, the a new mode that will only grow if the client's edge is not
already at a grow stopping point (ie against the edge of another window).

Created a 'GrowToFill' action. (Bug 3356)

Use the BSEARCH() macro in overlap placement

Currently the code rolls its own binary search, but now that we have
a well-tested binary search implementation in obt/ we can make use
of that.

Fix bugs and add unit tests for BSEARCH()

Also adds running unit tests to the release/go script so we can be
sure that all tests pass before we make a release.

Fix nits for overlap placement center option

Add the old <center> option for the placement policy. (Bug 5946)

Original commit messages:
. Reformat to move closer to house style
. Add center on top of leat overlap place algo
. Add sentinel value to edge arrays
. Use a Size instead of a Rect for a centering field
. Fix off by one bug
. Need to declare dx and dy
. Pass length of edge array instead of recomputing
. Fix missing open-brace in config.c
. Address the more trivial subset of danakj comments
. Revert "Remove now-unused config_place_center option."
  This reverts commit 5e282dae08be3b900e0337efa0fae8f3ffa92cd7.
. Remove reliance on sentinel value when scanning edge arrays
. Avoid need to initialize Size structure by removing it :)
. Clean up field expansion code somewhat
. Compress code further by using a structure for common args
. Fix search for next grid point
. Squeeze it even more by not using Size at all

Fix gcc warnings

Fix a typo causing submenuShowDelay to be ignored (Bug 6019)

Add window type to If as well

This one required some extra helpers so is separate commit

Allow matching class, name and role in If action

Add ForEach action which is like If but runs on all clients

Also adds a Stop action that lets you stop running, in case you only
want to run actions on the first match.

Remove the else-on-newline rule from HACKING

Add queries to If actions

This allows the If action to run queries against a client other than
the target of the actions being run, for example to check state on
the focused window while performing actions on another window during
focus cycling.

The syntax looks like

<action name="If">
  <query target="default">
    <title>FooBar</title>
    <maximized>yes</maximized>
  </query>
  <query target="focus">
    <desktop>3</desktop>
  </query>
  <then>
   <action name="NextDesktop"/>
  </then>
</action>

The above checks the client window that the actions will run on to
verify that its title is "FooBar" and that it is maximized. If that
is true, it also checks that the currently focused client window is
on desktop 3. If that is true also, then it runs the NextDesktop
action.

The target="" option can be set to "default" which uses the client
window that the actions will run on, or it can be "focus" which uses
the client window that is currently focused.

The <query> tag is optional, and the conditions inside the query can
be placed directly inside the If <action> tag, as they were before
this change. In that case, a default <query> tag is assumed with
target="default" which matches the previous behaviour.

Multiple <query> tags can be present, and they must all be true in
order to run the actions in <then>. If any one is false, the actions
in <else> will be run instead.

Update HACKING for brace rules

The rules in HACKING for braces are kind of unclear, so add some
more rules and provide examples too.

Add a note to add the version to the bug tracker for new releases.

Also important to update IRC and keep hydrated.

Update po files for 3.5.2

Add CHANGELOG for 3.5.2

Update version in configure.ac to 3.5.2

No library interfaces were changed since 3.5.1.

Maximized undecored windows have incorrect top border. (Bug 5996)

If the window has no border since the theme has bwidth=0, or because the
window is truly undecorated via MWM hints, then we should not allocate space
for the frame of the window which will just be empty pixels.

window.inactive.label.text.color should default to white.

Currently it defaults to itself, which is NULL, and causes a crash.

The spec http://openbox.org/wiki/Help:Themes#window.inactive.label.text.color
claims that the default is white.

In 98b02c6b60bbde2a5db026b3ee3e6e6dc44d1a92 we had this property being read
twice, once with the NULL backup and once with white. It seems that we kept
the wrong one, so just switching this to white now.

Fix a write out of bounds in splitvertical gradients (Bug 3612)

If the gradient has height 1, then y1sz is 0. We don't want to use the
first color and move the data pointer, since this will move it past the
end of the array.

We use congfigure.ac not configure.am