tests: refactor t/58-wm_take_focus to use wait_for_event

tests: lib/i3test: Remove open_standard_window, introduce open_window

open_window has a better API than open_standard_window. It uses named
parameters and supplies default values for everything you don’t specify. This
way, you can use every feature which X11::XCB::Window supports.

tests: complete-run: remove debugging messages

tests: complete-run: display time i3 took for starting up

tests: complete_run: directly use X11::XCB instead of ::Connection

This saves about 0.5s wallclock time due to not starting up Moose/Mouse.
This is worthwhile when you develop a new feature and you are often invoking
complete_run for one specific test.

tests: use wait_for_{map,unmap} to eliminate more sleep()s

tests: lib/i3test: provide wait_for_map and wait_for_unmap

These functions should be used instead of calling wait_for_event directly when
waiting for MAP_NOTIFY or UNMAP_NOTIFY

tests: implement sync_with_i3 and use it instead of sleep()

Also use open_standard_window() in a few more places where appropriate

lib/i3test.pm: reformat exports list

Implement the I3_SYNC client protocol

This is mainly useful for the testsuite. The tests can wait until i3 processed
all X11 events and then continue. This eliminates sleep() calls which leads to
a more robust and faster testsuite.

lib/i3test.pm: Don’t sleep(0.25), but wait until the window was mapped

This makes it faster and less racey

Merge branch 'master' into next

Merge branch 'fix-dragging-fullscreen'

Skip dragging of floating cons in fullscreen mode

Merge branch 'master' into next

Merge branch 'fix-close-focus'

Bugfix: Correctly revert focus to other floating windows when closing a floating window

Uncovered by the testsuite \o/

Merge branch 'master' into next

Merge branch 'fix-variables-cfg'

Bugfix: use bufcopy instead of buf when boundary checking (Thanks thomasba)

Also replace the useless (bufcopy + (next - bufcopy)) with next

Merge branch 'master' into next

Merge branch 'fix-variables-cfg'

A bit more boundary-checking when replacing variables.

Makes valgrind happy

Bugfix: Correctly split key/value when parsing variables (Thanks xeen)

update docs/multi-monitor to include the output names and refer to the force_xinerama config directive

mention the force_xinerama configfile directive in the userguide

Add force_xinerama configuration option

The configuration option does the same as the commandline parameter, except
it can be easily set by the user (e.g. you are using KDM and can't start a
session through ~/.xsession).

Signed-off-by: Michael Walle <michael@walle.cc>

Merge branch 'master' into next

Merge branch 'fix-i3bar-x11'

i3bar: Bugfix: Check if the X11 connection is unavailable

This fixes the condition where the i3 socket for some reason did not produce an
error, but the X server exited (earlier than i3?) and the left-over i3bar
process would consume 100% CPU.

How to reproduce the problem:
1) Start ./testcases/Xdummy :8
2) Start DISPLAY=:8 i3bar -s <socket path to i3 on :0>
3) Kill the Xdummy

Merge branch 'master' into next

Merge branch 'fix-focus'

Bugfix: Only change focus in tree_close() when the container was focused before

Fixes: #441
I’m not sure if that’s really all there is to it. Seems a bit too simple :).
We’ll see if I missed some corner cases in the next few days…

Merge branch 'master' into next

Merge branch 'fix-floating-move'

Bugfix: Avoid out of bounds coordinates when moving floating windows (Thanks eeemsi)

This commit makes the coordinates proportional when moving floating windows.
That is, if you have a window at the bottom of your 1920 px monitor and move it
to your 800 px monitor, it will be at the bottom of the 800 px monitor (and not
out of bounds).

Merge branch 'role-criterion' into next

docs/userguide: document the window_role criterion

t/65-for_window: add tests for window_role

Implement the window_role criterion (checks WM_WINDOW_ROLE)

Closes: #446
This is handy for matching specific windows of a multi-window application, for
example only Pidgin’s buddy list window.

tests: also make launch_with_config() log to LOGPATH

Merge branch 'master' into next

Merge branch 'fix-output-focus'

Bugfix: Use con_orientation(). Fixes switching between different outputs

Fixes: #518
For stacking containers, direct access using con->orientation is wrong, we need
to use con_orientation().

Merge branch 'i3-input-fmt' into next

i3-input: update manpage

i3-input: implement -F (format) option, deprecate -p (prefix)

This introduces the '-F format' parameter, which takes a format and replaces %s
in it with the user input. An example: The user should enter the target
workspace name. The appropriate i3-input invocation looks like this:

    i3-input -F 'workspace "%s"' -P 'Switch to workspace: '

i3-input: add missing \n in --help

i3-input: reformat main.c

Merge branch 'master' into next

Merge branch 'fix-floating-move'

fix-bug-515: always adjust floating position when moving to another output

Floating con position needs to be adjusted when moving to another
output, regardless of whether or not the target workspace is visible.

Merge branch 'master' into next

Merge branch 'fix-warp-same'

Bugfix: Don’t warp the pointer if it already is on the target output (Thanks cls, pnutzh4x0r)

My testcase was putting a floating window on the left output, but overlapping a
little to the right output. Then switch to a workspace on the right output.

Merge branch 'floating-focus' into next

tests: extend t/35-floating-focus to use focus left/right on floating windows

Remove obsolete code for floating focus

Implement focus switching (focus left/right) for floating windows

Fixes: #475

Also make compilation possible on systems with libev 3

From the source:

    We need ev >= 4 for the following code. Since it is not *that* important
    (it only makes sure that there are no i3-nagbar instances left behind) we
    still support old systems with libev 3.

document the dependency on libev >=4.0 for ev_cleanup (Thanks phnom)

libev 4.00 was released 2010-10-25, so nearly a year ago.

Merge branch 'master' into next

Merge branch 'fix-float-size'

Tests: fix floating size, add a test to check floating size when starting in tiling

Send the correct X11 border_width in faked ConfigureNotifys

Bugfix: Honor the X11 window border in ConfigureRequests

Bugfix: Honor the X11 window border when calculating dimensions for floating windows

Bugfix: Don’t use top border height twice when calculating dimensions for floating windows

fix build in 'next' (Thanks thomasba)

Merge branch 'master' into next

Merge branch 'fix-floating-confreq'

Bugfix: Correctly handle ConfigureRequests for floating windows in multi-monitor setups (Thanks thomasba)

Fixes #519

Bugfix: Don’t fix floating coordinates when reassigning (fixes flickering)

This fixes flickering when moving floating windows between different monitors.

Merge branch 'fix-reload-assignments' into next

Bugfix: Correctly free old assignments when reloading

Fixes #516

re-indent load_configuration()

update changelog (no, not releasing anytime soon)

Same bugfix as 2a215fd, but for assignments with invalid criteria

Merge branch 'master' into next

Merge branch 'fix-empty-criteria'

Bugfix: Ignore for_window commands with empty (invalid) criteria (+test) (Thanks aksr)

finally remove the deprecated 'screen' keyword from cfgparse.l

Merge branch 'new-assign' into next

docs/userguide: fix the for_window syntax

docs/userguide: document the new assign syntax

make the old assign syntax trigger an i3-nagbar warning (it’s deprecated), adjust test

Introduce a new syntax for the 'assign' command:

Instead of using a quoted string to specify the class / title, the assign
command now uses criteria, just like the for_window command or the command
scopes.

An example comes here:

    # Assign all Chromium windows (including popups) to workspace 1: www
    assign [class="^Chromium$"] → 1: www

    # Make the main browser window borderless
    for_window [class="^Chromium$" title=" - Chromium$"] border none

This gives you more control over the matching process due to various reasons:

1) Criteria work case-sensitive by default. Use the (?i) option if you want a
   case-insensitive match, like this:
   assign [class="(?i)^ChroMIUM$"] → 1

2) class and instance of WM_CLASS can now be matched separately. For example,
   when starting urxvt -name irssi, xprop will report this:
   WM_CLASS(STRING) = "irssi", "URxvt"
   The first part of this is the instance ("irssi"), the second part is the
   class ("URxvt").
   An appropriate assignment looks like this:
   assign [class="^URxvt$" instance="irssi"] → 2

3) You can now freely use a forward slash (/) in all strings since that is no
   longer used to separate class from title (in-band signaling is bad, mhkay?).

Kill left-over i3-nagbar processes on 'exit'

docs/userguide: provide an example of case-insensitive matching with PCRE (Thanks stfn)

Make the old 'assign' case-insensitive again (+test) (Thanks aksr)

Merge branch 'pcre' into next

pcre: enable UCP, UTF-8 (if available), extend t/19-match

mention PCRE in docs/userguide

debian: add libpcre3-dev as build-dependency

document the new dependency on libpcre in the DEPENDS file

Implement support for PCRE regular expressions for all criteria (for_window, commands, assignments)

extend t/19-match to also test regular expressions

i3-msg: include newline at the end of the reply

Merge branch 'clean-resize' into next

Restrict 'resize' command to left/right for horizontal cons, up/down for vertical cons

This makes the interface much clearer and avoids confusion about which key to
press in which situation.