This patch implements support for icons in user-defined menus into Openbox

Image loading is done using the Imlib2 library.
I chose Imlib2 because it's pretty fast, it's easy to use, supports many file
formats (tested xpm, gif, jpeg, png) and doesn't introduce too much bloat (it
depends :)).

I ported the patch to 3.4.7-pre3 and added some enhancements. Caching is much
better now, and icons can be disabled at compile time using --disable-imlib2
option.

What's new?
Syntax of configuration files (namely rc.xml and menu.xml) has been changed
slightly to allow users to associate icons to menu entries. This is done by
specifying path to icon file in the new "icon" attribute in "<item>" element,
e.g:
<item label="Vim" icon="/usr/share/pixmaps/vim-32.xpm">
  <action name="Execute"><execute>x-terminal-emulator -T Vim -e
vim</execute></action>
</item>

If user doesn't want to display any icons in his user-defined menus, he/she can
disable icons in rc.xml, inside "<menu>" section:
<menu>
 ...
  <showIcons>no</showIcons>
 ...
</menu>
Default value is "yes".
(New boolean variable "config_menu_user_show_icons" has been added to source
code.)

An icon is loaded (using menu_item_attach_icon()) when a new entry of menu is
created. Fortunately, I haven't notice any performance problems because of this
:).

Make focus_valid_target() take a desktop argument.

So you can find valid targets on desktops other than the currently visible one

use the valid_focus_target() method for choosing what to show in the client list menus

allow alt-tab etc to move focus to skip_taskbar windows when they are iconic

use obt_xml_* to navigate the xml file (also XML tags are case sensitive)

ignore some added tests

Don't handle input events on prompts if they should be used for a binding/menu instead.

Also, be more careful about making the prompt buttons look pressed, don't make them pressed from a motion notify event if they didnt first handle the press.

Merge branch 'backport' into work

Conflicts:

Makefile.am
openbox/actions/focus.c
openbox/config.c
openbox/event.c
openbox/menuframe.c

Fix a crash when moving into a menu on top of a separator and nothing is selected

Improved menu behaviour

Make 'left' key in a menu close it's visible child menu and move to the parent
  properly.
Make clicking on a menu item in a menu close any of its grandchildren and
  remove focus from its children, effectively focusing the menu item you
  clicked on.

Change the openbox website to openbox.org

add a manpage for obxprop

Add a --root option for obxprop

Allow infinite delay for submenu show/hide by using a negative value.

Some fixes for the submenu hide delay code

When nothing in a menu is selected, go back to selecting the open submenu.
Adjust the LeaveNotify event handling to only respond when there is not a
  EnterNotify coming for the same menu frame.
Change the default submenu show/hide delays.
Have the default values for submenu show/hide match the default rc.xml

improve submenu hide delay

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

This reverts commit 828c095c8b5a2df96a38faaeb8a0df504e68e70f.

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

This reverts commit 8ada991d829671ab5e6dd4c5526b6a0238ba6a16.

Revert "A fix for delayed hiding of submenus."

This reverts commit 1fb198410e9d3ca660d91d1049249db0f2f47732.

fix a broken if statement (bug #4464)

Make the Focus action stop all interactive actions (Fixes bug #4436)

Old < 3.4.8 Openbox versions would stop focus cycling when focus moved.  Now that is not the case, but you should be able to stop it by Choosing a window explicitly still.

Set the XDG_MENU_PREFIX to "kde-" when running an Openbox/KDE session.

reflect the change that parse.c/h was renamed to xml.c/h in the file header comments

when holding Shift for the client list menus, bring the selected client to the current desktop (Request #4284)

check more headers before including them

rename the obt_parse library to obt_xml (since it is very xml specific)

add obt_parse_save_file() method to the obt parse library

Set up work for making a 3.5 prerelease.

Set version stuff to 3.5.0-rc1.
Copy the CHANGELOG from 3.4-working.
Rename the obt-4.0 and obrender-4.0 pkgconfig stuff to obt-3.5 and obrender-3.5
Rename the "render" directory to "obrender" so that the public headers can be
  installed in <obrender/*>

Merge branch 'backport' into work

make a config option for the inactive osd font

Add themeing options for inactive osd labels, osd.inactive.label.*

(rename the current osd label options from osd.label to osd.active.label, similar to the window.(in)?active.label options.

Rename obprop to obxprop (Fixes bug #4419)

Avoids collision with openbabel.

A fix for delayed hiding of submenus.

It fixes this:
(Mikachu) dana: i think his issue was if you have a menu with two
          submenus s1 and s2, open the s1 submenu, then point to s2 and
          go into s1, then after the delay, s1 is hidden despite you
          being inside it still, because s2 was never unselected

Add a focus option, unfocusOnLeave that removes focus from a window when the pointer leaves it

This uses the same delay to unfocus as is used for focusing on enter

Merge branch 'backport' into work

Conflicts:

openbox/screen.c

Show the desktop popup as long as the interactive desktop switch is active

Allow delayed focus to work while in a menu or move/resize since it wont kill them now

Hide the focus popup if we change desktops and one of the windows in it disappears

Don't move focus away from a window if it was focused and it didnt hide when switching desktops

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.