Merge branch 'master' into next

Merge branch 'fix-assign-render'

Bugfix: Render workspaces created by assignments to use correct coordinates (Thanks meaneye)

Previously, i3 would send width=0, height=0 to windows which were put on
workspaces created by an assignment (that is, invisible workspaces,
which do not get rendered normally).

fixes #653

rendering: fix bottom line of tabbed borders decoration not continuous

fixes #814

Signed-off-by: noxxun <noxxun@gmail.com>

Implement variable border widths for pixel/normal

fixes #325

Display appropriate cursors when resizing or moving floating windows.

Clicking the root window should try to focus the relevant workspace.

Replace duplicate "__" workspace prefix checks with a single function.

draw empty title if WM_NAME is empty

fixes #811

add descriptive titles to split containers

testcase for propagating urgency

mark parents of urgent container also as urgent

Userguide: Added link from 7.2 to 6.6

The multiple monitors section (7) is probably the most
likely place for someone to go look for how to move
workspaces between monitors (6.6).

Close empty workspaces after cross-output move (+test) (Thanks chrysn)

fixes #795

remove unnecessary code in render_con().

The removed code was add by commit 61b8a62 to fix #564. That bug is cause
by rendering the mplayer window again as a floating window (even if it
has been rendered before as a fullscreen window, at line 202). So simply
checking for fullscreen window is enough to solve this problem. Treating
floating/tiling fullscreen window differently is not needed.

In get_output_next(), avoid an off-by-one for adjacent outputs and || mutually-exclusive failure conditions.

t/506-focus-right: also verify that focus up/down is a no-op (Thanks swh)

Make get_output_next() work with non-aligned RandR setups (+test) (Thanks Feh, swh, Moritz)

A good visualization of the new algorithm is this:

           +--------+
           |        |
+--------+=|   S1   |========================
|        | |        |
|   S0   | +--------+
|        |         +--------+
+--------+=========|        |================
                   |   S2   | +--------+
                   |        | |        |
                   +--------+ |   S3   |
                              |        |
                              +--------+

When focus is on S0, 'focus output right' will first match S1 (the
closest output which overlaps in the highlighted area), then S2, but not
S3 (since S3 does not overlap into the highlighted area).

fixes #669
fixes #771

i3bar: Honor "urgent" protocol hint by unhiding

i3bar: Allow to force unhide with draw_bars

i3bar: Allow child to specify signals to use

We now wait for the child process to send the first line before stopping
it to use the signal which might be specified in the i3bar protocol
header
Since clients might use the same signal for both stop and cont, we also
save the stopped state of the child to avoid stopping it while hidden!

i3bar: Fully parse the JSON header

i3bar: Introduce i3bar_child struct

i3bar: Handle the first line with another callback

i3bar: Split JSON line logic to read_json_input

i3bar: Split flat line logic to read_flat_input

i3bar: Split stdin reading logic to get_buffer

i3bar: Rename determine_json_version to parse_json_header

docs/i3bar-protocol: Document stop/cont_signal

i3bar: Rework unhide/hide on workspace urgency

We now check globally for workspace urgency instead of per-output since
the result is the same but we call unhide_bars/hide_bars only once this
way

tests: Bugfix: use exit_gracefully() in t/200-urgency-timer

Make "[move] workspace number" accept a default ws name after the ws number

cfgparse: be forgiving about a missing "ms" after a duration

While it’s certainly better and clearer to specify it, we should do the
right thing when the unit is missing, just like CSS for example
(margin: 0; is okay, margin: 0px; too).

tests: rename double t/198, add boilerplate

tests: fix t/113-urgent.t after previous commit

implement delayed urgency hint reset

If there is a client with an urgency hint on another workspace and
switching to this workspace would cause the urgency to be reset (by
moving the focusing to the client), delay the reset by some time. This
gives the user the chance to see it.

This commit adds the possibility to configure the urgency delay timer
duration using the 'force_display_urgency_hint' directive. Also,
documentation and a testcase was added to allow for automated checks of
the intended behavior.

fixes #482

fixes #776

this implements both the "move container to workspace back_and_forth" command
and movements to the same workspace when auto_back_and_forth is set.

it includes documentation and test suite additions by michael.

it also simplifies the workspace_show_by_name function (making use of
workspace_get accepting NULL pointers).

don’t use reserved identifiers for include guards (left-overs)

fixes #804

tests: add testcase for the 'mode' IPC event

tests: add missing boilerplate

Add new subscribe event 'mode' for binding mode changes

Introducing a new event to subscribe called mode. It's fired up
when i3 changes binding mode (like switching from default to resize).
IPC guide adjusted also.

add proper error handling for in-place restarts (Thanks Markus)

fixes #806

Use ev_signal to avoid async-unsafe functions (Thanks Markus)

Functions such as fprintf() might be unsafe to use in a signal handler,
see http://stackoverflow.com/questions/3941271/#answer-3941563

By using ev_signal, libev will use a tiny signal handler which just
passes on the information and then calls (outside of the signal handler)
our callback function which can use fprintf() and other unsafe
functions.

fixes #803

don’t use reversed identifiers for include guards (Thanks Markus)

Done with:

    sed -in 's/\(ifndef\|define\) _\([0-9A-Z_]*\)$/\1 I3_\2/' include/**/*.h

fixes #804

complete-run.pl: Check for missing executables

Merge branch 'desktop-file'

Merge branch 'master' into next

Fix xsession.desktop to comply to fdo standard

For more information please see:
http://standards.freedesktop.org/desktop-entry-spec/desktop-entry-spec-latest.html

Merge branch 'remove-ev'

Merge branch 'master' into next

remove superfluous ev.h include statements

Merge branch 'master' into next

Merge branch 'tiny-fixes'

clean old and new libi3 (Thanks SardemFF7)

spelling fix: s/succesfully/successfully/

update debian packaging

Merge branch 'next'

Merge branch 'release-4.3' into next

fix make clean targets

man: set version to 4.3

man: update i3(1)

--help: note that nVidia’s driver supports RandR from 302.17 on

release notes: update and final touches

userguide: mention the FAQ

userguide: make moving cons/workspaces to other outputs its own section

Frequent requests for this functionality on IRC made me aware that it’s
not easily findable.

hacking-howto: fix wrong anchor to #_tree_reply

Allow changing the layout of workspaces by storing it (Thanks mhcerri)

Before commit 4976fa33504907510f1ee9e44884c116af0d801b, setting the
layout of workspaces to something else than the default would just mess
up the parent container of the workspace (the content container).

After that commit, it would create an unnecessary split container when
you change the layout _before_ opening any containers. To avoid this, we
now store the layout (similar to how the 'workspace_layout'
configuration directive works) and apply it when the first container is
attached to the workspace.

Fixes #796

Merge branch 'master' into next

Merge branch 'fix-fullscreen-scratch'

toggle fullscreen on 'scratchpad show'

If the focused application is in fullscreen mode and 'scratchpad show'
is executed, toggle the fullscreen mode before showing the scratchpad
application. Otherwise, the scratchpad app would not be visible anyways
but focus would be stolen.

fixes #613

Merge branch 'master' into next

Merge branch 'fix-scratchpad-focus'

Bugfix: Don’t focus the wrong workspace when moving to scratchpad (+test) (Thanks loblik)

The problem was that scratchpad_move() didn’t check whether the source
workspace was focused. Therefore, 'move scratchpad' only worked reliably
interactively , but not when used with criteria.

update release notes to reflect the current i3-nagbar situation

Only launch i3-nagbar for parse errors

Remove support for resize increment size hints for tiling windows

As discussed on the mailing list and the bugtracker.

fixes #540

sighandler: use non-existing backtrace filenames ($TMPDIR/i3-backtrace.%pid.%idx.txt)

sasprintf() already handles errors, we don’t need to do that twice

sighandler: provide gdb with pipe stdin/stdout fds (necessary for gdb < 7.5)

Replace exit with backtrace in crash dialog

When the user hits 'b', attach gdb and dump a backtrace to the tmpdir

include the testcases in the dist tarball (required for building docs)

docs/testsuite: add pointers to additional docs

add boilerplate to all testcases with documentation references

fix typo

debian: add lib-i3test.html and lib-i3test-test.html to docs

tests: document each and every i3test function

tests: move does_i3_live to i3test::Test

tests: make cmp_float a real test instruction

docs: generate HTML from testsuite POD documentation

use pwd -P instead of readlink -f (Thanks Marcus Crestani)

readlink -f does not work on Mac OS X

build fix: remove src/cmdparse.* in make clean

This was removed in commit 8853334bbefd57f8f9bd6bdeb0c221616b40b446

don’t errnously detect --release bindings as duplicates

docs/userguide: document the --release flag for key bindings

docs/userguide: use --no-startup-id for executing shell scripts

ignore modifiers for KeyRelease bindings

For the following binding:

    # Simulate ctrl+v upon pressing $mod+x
    bindsym --release $mod+x exec --no-startup-id xdotool key --clearmodifiers ctrl+v

you can now use either:
1. press $mod, press x, release x, release $mod
2. press $mod, press x, release $mod, release x

fixes #485

naive implementation of 'bindsym --release' (and bindcode)

The implementation is naive because the user has to generate exactly the
event he specified. That is, if you use this binding:

    bindsym --release $mod+x exec import /tmp/latest-screenshot.png

Then it will only be triggered if you hit $mod, hit x, release x,
release $mod. It will not be triggered if you hit $mod, hit x, release
$mod, release x. The reason is that the KeyRelease event in the latter
case will not have the modifier in its flags, so it doesn’t match the
configured binding.

re-indent src/config.c

update release-notes

rendering: ceil() instead of truncating for tabbed deco_rect width (Thanks szalik)

In tabbed mode, the available width (say 1280) is divided by the amount
of child containers (say 3). Before this commit, we just truncated the
result and would end up with 426 + 426 + 426 = 1278 pixels that we
render to. Now we render a bit too much, but that’ll at least not give
us graphics corruption on any side :).

fixes #791

fix format string warnings