Merge branch 'backport' into work

Make the relative desktop switching actions interactive

The desktop popup can now be shown "permanently" in which case it stays until
you tell it to hide.

Ranges of 0 width cannot intersect (Fixes bug #3717)

Allow the interactive cancel function to be optional

Add some directional focus bindings to the default config file.

Make it possible for an action name to choose whether it is interactive or not based on its options.

This way we can use the same name with options for an interactive action and a
non-interactive action.

Shorten the names of the ObActionsInteractive* functions to ObActionsI*

Add a ObActionsIPreFunc that is called for interactive actions
before the interactivity (key/mouse grab) is started.

Add a ObActionsIPostFunc that is called for interactive actions
after the interactiviti (key/mouse grab) has ended.

add a test that iconifies a window after a short delay

Calculate the work area using other already existing and better maintained and trusted code

Fix resistance against struts, the resistance was always 0

Merge branch 'backport' into work

Conflicts:

obt/keyboard.c
obt/keyboard.h
openbox/event.c
openbox/menuframe.c
openbox/moveresize.c
openbox/openbox.c
openbox/screen.c

Fix the delayed hiding of submenus, so that they don't end up hidden while the entry is selected

Ignore enter events caused by closing openbox menus

Comment out some debug messages for the image cache

Make the NET_WORKAREA hint work the way it used to, across all monitors

The spec is not going to change this hint, but rather add a new one. So for now
just revert to the old behaviour.

Don't hide submenus immediately when unselecting the parent's entry

This allows users to move to the submenu across other menu items (the same
as they already could across other menu items that were submenus).

This uses the same config delay for hiding submenus as it does for showing
new ones.

Based off the ideas in bug #3762.

Let menus place themselves on monitors where the mouse is not present

This fixes a bug which forced menus to show up on the same monitor as the
mouse pointer.

Allow the user to bind more than one keycode to a keysym for Ob Menus/Move/Resize

If the user has escape bound to more than one keycode then they can use any of
them to close a menu.  This change applies to the hardcoded keys in openbox,
which are used for the menus and for move/resize, and maybe other places.

Merge branch 'backport' into work

Add an obprop tool which can print UTF-8 strings properly, for our users.

In the applications section tell the user to use _OB_* properties for the name/class/role, and refer to the still-non-existant obprop tool.

Merge branch 'backport' into work

Conflicts:
openbox/config.c
openbox/event.c
openbox/prop.c
openbox/prop.h
openbox/screen.c
openbox/screen.h

Set hints that remember the WM_WINDOW_ROLE and WM_CLASS properties for our users.

The hints are _OB_ROLE, _OB_NAME, and _OB_CLASS.

Get the session client id from new gnome-session

See http://live.gnome.org/SessionManagement/GnomeSession#A1._Launch

Gnome-session sets the DESKTOP_AUTOSTART_ID env variable with the SM client id
instead of passing it on the command line.

Run Openbox with gnome-session the proper way, without using the deprecated gnome-wm.

See https://bugzilla.gnome.org/show_bug.cgi?id=555406#c5
Hopefully this continues to work for a while.

Make sure popup dialogs are not waiting to be shown after they are freed

Add an openbox.desktop file for the applications/ directory

This is for gnome-session to start Openbox correctly as the window manager
See the commit log for http://cvs.fedoraproject.org/viewvc/devel/openbox/openbox.desktop?revision=1.3&view=markup

Don't need 2 exit actions in the default menu now, they both would do the same thing

Remove desktop hints since Openbox won't overwrite them, but they may be set from GDM or other sessions

Fix a typo in xdg-autostart

Make the desktop popup show on the primary monitor as defined in the config.

This matches the behaviour of the focus cycle and key chain popups.

Revert "Show desktop switch popup on every monitor"

This reverts commit 2e1adce628ee3234accc5d88cafb57672800cae0.

Revert "Some fixes for the pager popups, and avoid a crash if you destroy a popup while it is going to be shown later by a delay."

This reverts commit 33328583a143677d27eb3d081ce66532c3aaca1c.

Add a primaryMonitor config option, where the focus-cycle and keychain popups will appear

Bug in choosing the active monitor when doing a move/resize

Make the edge detection fully include monitor edges.

This fixes the move-to-edge behaviour, when moving past the edge of a monitor,
the window will stop with its tail edge against the inside of the monitor's
edge.

Make the default placement on the active monitor (Any is madness)

Find a valid image to use in the image cache. Fixes bug #1149

The larger of the width and height has to match.  If the smaller matches, then
it would have to be resized down to fit inside the area, so that does not
count.

Properly react when a client's strut changes.

Previously it would only react if the height of the strut changed, not if its
start/end changed (that was a long-standing bug).

Fix for rendering RGBA and Image textures.

RGBA and Image textures could exceed their tarea if given an x or y offset
inside the area that is > 0.

Fix the Focus/Activate actions to focus windows on other desktops correctly

When moving focus across desktops, make sure Openbox doesn't think that a window which is no longer visible is still focused.

The colormap is being set twice when moving focus to the root window

Add a focus debug message for the frame's visible focus state

Fully maximized windows still have a normal titlebar.  Fixes bug #4373

Also, maximized windows were having their client padding reduced by the size of
the outer border from this same bug, which is now fixed.

Add a WM_CLASS to the skiptaskbar test

Allow skip_taskbar windows to be focused on map when the user requests it in rc.xml. Fixes bug #4350

Set a minimum value for the screenEdgeWarpTime for 25ms.

When this is very small it just gives X/Openbox a heart attack and ends up going forever.  Even 25 is quite too fast to be usuable so it should be a good minimum.

Use test -r instead of test -e for solaris.  Fixes bug #4253.

Fix stacking for transients vs helper windows. Fixes bug #3851

This allows transient windows to be above helper windows.  And generally keeps
helper windows below transients, unless they are raised directly.

Make openbox-gnome-session work with gnome-session > 2.22

Follow up for commit 7f36e21ea9d86df5e6fa62d2888891ed957c4639

The actions/session.c file was removed.

Merge branch 'backport' into work

Conflicts:

openbox/actions/all.h
openbox/actions/session.c
openbox/client.c
openbox/event.c
openbox/grab.c

Don't hilite new windows when we're restoring them from a saved session.

Fix client_activate() to work for internal Openbox menus

client_activate() is a helpful way to focus a window on another desktop,
but only Openbox is allowed to do such things, user messages cannot.

Don't kill keygrabs when focus moves.

Applications should be better behaved by now, and GTK based apps seem to be
at least. We can file bug reports with them if we need to still.

Change _net_active_window behaviour.

Move focus on _net_active_window "app" requests if focus stealing would be
allowed. If focus is not given to the target window and it is hilited (or
any other situation where it is hilited) and the window is on another desktop,
then also raise it and make it the LRU window, so when you switch desktops
you go right to it.

Add needed linebreaks to the end of some debug messages

strip leading/trailing whitespace off stuff when reading it from the configs

Combine the Exit and SessionLogout actions

Just "do the right thing" based on if you are connected to a session manager
or not.

Merge branch 'backport' into work

Conflicts:
openbox/popup.c

Don't ever give borders to windows that didnt have them before by making them "undecorated" with the keepborder config option on.

This fixes what commit 85f39cd27e7ea0eec8bc78f6139092b44fda2dad fixed for the
fullscreen window case, but also fixes bug #4344.

Add a comment for commit ec9fbf05df3ac807d60ae6af36236ffc6567bbdc regarding struts with multiple monitors.

Some fixes for the pager popups, and avoid a crash if you destroy a popup while it is going to be shown later by a delay.

Fix an unsigned/signed warning.

Fix some signed/unsigned warnings (and keep stuff within 80 columns).

Keep things in 80 columns

Pointers are long-sized on x86_64.

Fix spacing from 8f37a97a4cc29d35f6a632ef2f71ac760e301ee5

Add a test that grows a window by changing its normal hints.

Revert "Revert "make windows stay on screen when..."

Just kidding.  This doesn't revert d81d54c7a977a034199f24fc8fa7433e634aaf91 but
it redoes the work that patch reverted in a better, less intrusive, way.

Revert all commits for the Hooks feature to move it to a topic branch.

This reverts commit fd2f617be7ff57fcb187daa737f66d243544c8cd.
This reverts commit a47d0a53652a96ca1df96fc9268757df1431ae55.
This reverts commit a1908e076736e1a618bd305c5963dbfecaa30497.
This reverts commit dc2e6f6bf7143a56de360a393b33906735e63625.
This reverts commit 985e7dadf9a3ebf4bd265d955c3198e96405e5d2.
This reverts commit db781556d63d1a50bd1b1b4b6b5423ef703bf2c7.
This reverts commit feec8f663f0a11546c2da87575fecc8a88d97ca1.

Make clang happier

Add asserts to default: in switch statements
Store pointed to variables locally so it knows they don't change
Remove some dead assignments
Mark ob_exit_with_error as noreturn
Use "%s", msg instead of just msg to printf style functions
Use the c_pfocus variable

Merge branch 'backport' into work

fix a crash in the SessionLogout action

Merge branch 'backport' into work

Respect $XDG_CONFIG_HOME in openbox-session

Reload motif wm hints when property changes

Google's chrome does this when you toggle window decorations.

Based on patch in #4250 by Daniel Erat.

Add all the action names used in 3.4 so configs don't break

Merge branch 'backport' into work

Conflicts:
openbox/actions/desktop.c
openbox/client.c
openbox/event.c
openbox/extensions.c
openbox/popup.c
openbox/screen.c
parser/parse.c

Update slovak translation.

Remove newline from startupnotify description

It doesn't look good in some places, e.g. WnckTasklist (gnome-panel)

[ Also update translations -- Mikael ]

Don't replace ~ when we shouldn't

Bump glib req to 2.14 to get GRegex, now we only replace ~ with your
homedir in an action if it is (at the start of the string or preceded
by whitespace) and (at the end of the string or followed by (whitespace
or a slash)). (?:^|(?<=[ \t]))~(?=[/ \t$])

Fixes bug #4033.

Another debug message typo

Allow escaping _ in menu labels by putting __

Currently you can't mark anything that comes after the __ with _ to make
that a shortcut.

Don't trust xinerama not to return NULL

We check that xinerama is active already, but someone got a NULL here.

Update ukrainian translation and rename to uk.po

Don't draw borders in fullscreen mode when keepBorder is on

Fix interpretation of struts with multiple screens

According to the WM Specification, the left, top, right, and bottom
fields are to be declared relative to the overall X screen dimensions,
not the monitor dimensions.

The example given in the spec (v1.3 or 1.4draft2) is: "Another example
is a panel on a screen using the Xinerama extension. Assume that the
set up uses two monitors, one running at 1280x1024 and the other to the
right running at 1024x768, with the top edge of the two physical displays
aligned. If the panel wants to fill the entire bottom edge of the smaller
display with a panel 50 pixels tall, it should set a bottom strut of 306,
with bottom_start_x of 1280, and bottom_end_x of 2303. Note that the strut
is relative to the screen edge, and not the edge of the xinerama monitor."

In my case, I have a 1680x1050 monitor to the left of a 1920x1200 monitor
aligned at the top. I then have a gnome-panel along the bottom edge of
the 1680x1050 monitor with a height of 24 pixels.

xprop reports the following partial strut: _NET_WM_STRUT_PARTIAL(CARDINAL)
= 0, 0, 0, 175, 0, 0, 0, 0, 0, 0, 0, 1679 which is correct according to
the spec. Gnome-panel is reserving the 150 pixels along the bottom that
aren't visible on the screen plus the 25 it requests for itself.

However, maximizing a window on this monitor leaves a gap of exactly 150
pixels between the bottom edge of the maximized window and the top edge
of the panel.

Also, when the 1680x1050 monitor is the primary monitor (id=1) then the
_NET_WORKAREA property on the root window is also off by 150px for the
same reason.

This patch fixes the two issues I mentioned for exterior monitor edges.
It doesn't attempt to account for "interior" monitor edges (i.e. a 'left'
strut on monitor A when monitor B is directly to the left of monitor A)
because it's not possible to do so with the current strut specification
(see http://mail.gnome.org/archives/wm-spec-list/2004-March/msg00004.html
for a discussion on this limitation)

This could be avoided by having the partial strut atom contain a xinerama
screen ID that the strut applies to, but unfortunately the discussion
all those years ago never got anywhere.

[ quoted from bug #3792 ]

Fix a typo in debug message

Show desktop switch popup on every monitor

This should be a satisfactory fix for #3694 I hope.

Give the popup to the timer remove so it does the right one

Only the chroot popup uses delayed mapping so this is just a latent bug.

Fix memory corruption when y2sz is 0.

data is incremented one too many times when y2sz is zero, leading to
memory corruption.

[ also changed % 2 to & 1 -- Mikael ]

Add "prev" and "next" as possible targets for moveto/resizeto actions.

One of the Debian users asked if it's possible to send a window to other
monitor when using xinerama, especially useful of you have 2 monitors and want
to toggle a window to the other one. I wrote a patch that implements next and
prev to also make that work for 3 or more workspaces.

Only open the default font once, then ref it.

As suggested in #3622, we don't need to open the default font for every
place that wasn't specified in the theme. Solved a bit differently than
the patch given there.

client_validate should return FALSE only for UnmapNotifies that will cause the window to become unmanaged

Show window role in the debug message for name/class too.

Fix for #3715, app settings applied too late.

This caused problems for placing windows with decor turned off, the
placement code thought they had it on.

Add support for 24bit

Make at least the default configs validate

More things need to be case insensitive and whatnot.

Update and reactivate croatian translation.