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

Start on the second entry...

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

Merge branch 'mikabox/maybe' into wip/mikabox

Conflicts:
openbox/client_list_combined_menu.c

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.

Merge branch 'work' into wip/mikabox

Conflicts:
openbox/event.c

Merge branch 'backport' into work

Conflicts:
openbox/event.c
openbox/frame.c

Add Plural-Forms header to .po files that need it.

Handle InputShape type shaping too.

Many composite apps use this to pass through clicks where they are transparent.

Typos in error message from gdm-control

install-sh should be executable

Apparently on solaris, you can't invoke -x scripts via "sh install-sh".

Merge branch 'backport' into work

Only update the focus cycle indicator when already visible in frame.c to avoid overriding bar=no

Only update the focus cycle indicator when already visible in frame.c to avoid overriding bar=no

Another problem pointed out by clang

Using format specifiers without arguments in printf is still bad.

openbox/debug.c:105:18: warning: format string is not a string literal (potentially insecure)
    fprintf(out, log_domain);
                 ^~~~~~~~~~
openbox/debug.c:107:18: warning: format string is not a string literal (potentially insecure)
    fprintf(out, level);
                 ^~~~~
openbox/debug.c:109:18: warning: format string is not a string literal (potentially insecure)
    fprintf(out, message);
                 ^~~~~~~

Merge branch 'backport' into work

Conflicts:
openbox/event.c

Use ngettext for %d desktop(s).

This poses a small problem. We currently let translators reorder this
string, but ngettext only takes one numeric argument. This means that you
can either get correct pluralization or the order you want, but not both.

I fixed up the languages I understand at a very basic level, but the
rest will need translator assistance.

Fix a problem pointed out by clang

openbox/client.c:269:10: warning: incompatible pointer types passing 'gint *', expected 'guint32 *'
    if (!OBT_PROP_GET32(self->window, NET_WM_USER_TIME, CARDINAL, &user_time))
         ^~~~~~~~~~~~~~

Merge branch 'work' into wip/mikabox

Conflicts:
openbox/event.c

Merge branch 'backport' into work

Don't move windows to current desktop on net_active_window if the event didn't come from the user.

Instead, just activate the flashy thinger.

Using format specifiers without arguments in printf is bad.

Add GroupMembers action.

Similar to AllClients, but acts on all members of the group the current
action target is a member of.

Merge branch 'work' into wip/mikabox

Conflicts:
openbox/event.c

Merge branch 'backport' into work

Bring helper windows when following a window.

_NET_WM_USER_TIME=0 means don't focus the new window.

Merge branch 'backport' into work

Conflicts:
openbox/client.c
openbox/prop.c
openbox/prop.h

No \n on ob_debug in this branch.

The code to replace $wid/$pid died on $foo.

It failed to increment a variable after finding a $ but no pid or wid,
which led to a fun infinite loop. Also don't check if c[1] is w if we
already established it was p.

Show name/class when mapping in debug mode.

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

This reverts commit 73adc6480d59fc5dccf9fe19ed5234c9582b328c.
I know I had a good reason for reverting this commit, I don't know what
that reason was though.

Correct a 64-bit bug in event_time_after

The code assumed the timestamps had the same domain as the type Xlib
uses for them, which is almost never the case with Xlib. Change all
involved variables to guint32.

give the compiler hints about what can be kept in registers so that it can optimize the gradient code better with all the pointers going on

Make sure the _MOTIF_WM_INFO atom exists.

Without this, urxvt won't use motif hints for borderless mode.

Update Basque translation.

Merge branch 'mikabox/actions' into wip/mikabox

Conflicts:
openbox/actions/focus.c
openbox/event.c

Add "locked" to if action

Disallow some more stuff on locked windows

Merge branch 'mikabox/maybe' into wip/mikabox

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

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

0 is the first monitor here, not 1.

Merge branch 'imlib-submenus' into wip/mikabox

Unrelated to this topic but whatever. Refactor and comment and note some bugs that should be fixed.

Merge branch 'imlib-submenus' into wip/mikabox

Is this better or worse?

XXX TODO BLAG ETC Allow icons also for submenus

This has some code that should be "refactored" a bit.

Merge commit 'nightm4re/3.4-working-padding' into wip/mikabox

Check both window+margin and just window for grow/move.

Merge commit 'nightm4re/3.4-working-padding' into wip/mikabox

Merge commit 'nightm4re/3.4-working-padding' into wip/mikabox

Mika says rearrange it!

Merge branch '3.4' into wip/mikabox

Needless externs.

Added <windowMargin> config item and made the margin thing respect it!

Silly commentating.

Move now has resistance.

Resist edges working with margin.

GrowToEdge functions respect windows having a margin the same as the desktop margin.

Add danish translation.

Merge branch 'buttons' into wip/mikabox

Conflicts:
openbox/client_menu.c
openbox/framerender.c
render/render.c
render/render.h
render/theme.c
render/theme.h

Properly escape strings before writing XML.

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.

No \n on ob_debug in this branch.

Show name/class when mapping in debug mode.

Merge branch 'work' into wip/mikabox

The code to replace $wid/$pid died on $foo.

It failed to increment a variable after finding a $ but no pid or wid,
which led to a fun infinite loop. Also don't check if c[1] is w if we
already established it was p. The code should probably check the length
of the string instead of reading random memory, but how fun is that?

Openbox doesn't support _NET_WM_USER_TIME_WINDOW atm.

Update catalan translation.

Add a translators note about not translating the word "SessionLogout" in an error message.

Add turkish translation.

Do focus menus with the mouse.

Update italian translation.

Update spanish translation.

Add Lithuanian translation.

Merge branch 'work' into wip/mikabox

Revert "make windows stay on screen when they are resized due to their normal hints changing"

This reverts commit 73adc6480d59fc5dccf9fe19ed5234c9582b328c.

Don't translate action names.

Merge branch 'backport' into work

a 64-bit fix

give the compiler hints about what can be kept in registers so that it can optimize the gradient code better with all the pointers going on

Update russian translation.

Allow to use a pango font description string.

Merge branch 'wip/edges' into wip/mikabox

This & shouldn't be here.

Merge branch 'imlib2' into wip/mikabox

Conflicts:

Makefile.am
openbox/config.c
openbox/menu.c

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
:).

Bump.

Merge branch 'work' into wip/mikabox

Conflicts:

openbox/moveresize.c

Merge branch 'backport' into work

Conflicts:

openbox/config.c
openbox/keyboard.c
openbox/moveresize.c

Fix typo introduced a few commits ago that made resizing windows hard.

Crazy focus model.

Don't need to explicitly check for xft.

Bump and changelog.

Missed a pair of parentheses when I resolved a merge conflict. You can now resize windows again.

Bump version to 3.4.7.

Remove an item from CHANGELOG as I just reverted that commit.

Merge branch 'backport' into 3.4