Merge branch 'master' into next

Merge branch 'fix-split-indicator'

Bugfix: force rendering when the parent’s orientation changed

Otherwise, the split indicator might not be refreshed even though it
should be.

fixes #858

docs/userguide: s/alt/Alt/

Implement smart popup_during_fullscreen mode

With this commit, the default behavior is to display popups while there
is a fullscreen application only if the popup belongs to that
application (as determined by the WM_TRANSIENT_FOR hint which
applications have to set properly).

fixes #663

spelling error: s/implementaiton/implementation/g

keycode default config: s/bindcode/bindsym (Thanks Tim)

docs/userguide: use $mod consistently (Thanks Conley)

Skip floating windows in the focus stack when moving through the tree

Includes a test for the new behaviour

Focus windows when middle-clicking (X paste)

As with most click-based focusing, this only has an effect when
focus_follows_mouse is disabled.

Skip floating cons in focus (child|parent) and stop them from being split

Focusing child from a workspace should now skip over the floating con and
go directly to its child. Focusing parent from that grandchild should leave
the workspace focused again.

bugfix: don’t send workspace command when at beginning/end of workspaces (Thanks whitequark)

fixes #843

exit with a proper error message when there are no outputs available (Thanks flo)

fixes #842

config parser: make newlines okay before opening braces (Thanks aksr)

Bugfix: Actually set border width in config_directives.c (Thanks strcat)

Allow 'focus $dir' to move out of non-global fullscreen containers

Allow workspace contents to be moved if there are only floating children

Fix fullscreen focus bug and corresponding test flaw

As the workspace might be reached via recursion (e.g. moving from the edge
of a fullscreen split container), it's necessary to check for a fullscreen
container whenever a workspace is reached.

Grab keys with all permutations of lock and numlock

This should prevent all cases of caps lock (or shift lock, on some
keyboards) from interfering with i3 key bindings.

tests: fix error messages, add 'bind' as synonym to mode blocks

Fix warning: exclude NULL parameters from format string (Thanks knopwob)

accept ctrl as synonym of control (Thanks SardemFF7)

error out instead of accepting invalid key bindings (Thanks SardemFF7)

add missing include/config_parser.h (Thanks slowpoke)

bugfix: config-parser: bind is a synonym for bindcode

fix warning about printf() field length

add test.config_parser to .gitignore

use the new parser by default

you can force the old parser with the command line flag
--force-old-config-parser-v4.4-only (which will be present in v4.4 only,
as the name suggests)

move owindow definition into the command parser

link the parser test binaries with -g

change the commands_parser prefix to 'command' for consistency

t/201-config-parser: test more directives, test error handling

port the entire old config parser to the new one

config_parser: implement <number>s, proper error handling

tests: use new assign syntax, drop legacy test

generate-command-parser: support <number>s, state ID replacing and…

…determining the next state within a function like cfg_criteria_pop() by
passing next_state in the ConfigResult (or CommandResult) and using it
after calling.

add first bits of a (custom) config parser

generate-command-parser: make input/output configurable

i3-msg.man: fix reference to the "reply section" (Thanks slowpoke)

allow floating cons to be reached using 'focus parent'

I suppose this was just an oversight. Let’s see if it causes any issues.

fixes #831

Improve startup sequence termination conditions

If a window with _NET_STARTUP_ID set is moved to another workspace, it
will delete any associated startup sequence immediately. This will also
occur if a window has a leader with _NET_STARTUP_ID set, if the leader
has no container (never been mapped).

A startup sequence may also be deleted if it's matched by
startup_workspace_for_window() and its 30-second timeout has elapsed.

Replace the discrete 'split' Con property with a simple function.

Un-fullscreen as needed when moving fullscreen containers

This avoids a case where a fullscreen container could be moved onto a
workspace that already had its own fullscreen container, leading to
two fullscreen containers on top of each other.

shmlog: Remove O_TRUNC flag for shm_open, we truncate ourselves

release-notes: s/mod+l/mod+e/ (Thanks hax404)

docs/ipc: add a warning to use an existing library (Thanks slowpoke)

docs/ipc: remove unnecessary newline (Thanks Merovius)

Merge branch 'fix-ws-layout'

Merge branch 'master' into next

set workspace_layout in create_workspace_on_output

fixes #835

Focus the relevant workspace when clicking any container.

Maintain relative positioning when moving floating windows between outputs.

testsuite: add 'new-test' helper script

add test for previous commit

Only re-focus the workspace when moving a con if the target ws is hidden.

Implement moving workspaces as if they're regular containers

Merge branch 'master' into next

Merge branch 'fix-floating-move'

Bugfix: Correctly move floating windows to invisible workspaces cross-output (Thanks swh)

Bugfix: with one ws per output, don’t crash on cross-output moves (Thanks moju)

fixes #827

Make the resize command honor criteria (Thanks Tblue)

fixes #816

Merge branch 'fix-indicator-pos'

Merge branch 'master' into next

Bugfix: Draw h-split indicator at the right position

fixes #817

Fix 'border toggle' (it "skipped" 1px border) (Thanks joepd)

fixes #818

Merge branch 'fix-floating-move'

Merge branch 'master' into next

Bugfix: don’t crash when dragged floating window closes (Thanks darkraven)

Previously, while the loop would not be executed anymore, the callback
(for actually repositioning the window after a MotionNotify) would still
get called once. This commit avoids that, thus fixing the crash.

fixes #819

userguide: document how to "un-scratchpad" a window (Thanks knopwob)

fix crash: send non-floating window with floating parent to scratchpad (thanks pkordy)

Fix a crash that occured when moving a window to the scratchpad that is
seemingly floating to the user, but actually a descendant of a floating
parent con (and itself non-floating). If that is the case, move the
floating parent container to scratchpad instead of the window.

fixes #740

Revert "raise floating windows when focusing (Thanks Marcos)"

This commit breaks floating window keyboard focus order
(t/135-floating-focus.t) when you have > 2 floating windows. Since
keyboard focus is more important than saving one click to raise floating
windows, I revert the commit.

Note that we cannot implement this without keeping a third list (beneath
floating_windows and focus) for the z coordinate of a floating window.
This seems not worth it.

This reverts commit 064be457e59603e1d4a4b45b4bf138dcc231a849.

raise floating windows when focusing (Thanks Marcos)

add testcase for commit 87525ad

remove async-unsafe functions from signal handler

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

This makes our signal handler useless and leads to infinite SIGSEGV
loops because the ev callback handler gets called only from within the
event loop, and control doesn’t necessary get to the event loop…

This reverts commit 514265b529ac78b7778eeee2db3dddb6f3a1c24c.

fix crash: urgent floating con on separate workspace (thanks Piotr)

If there is a single floating con on a separate workspace that is not
focused, and this con becomes urgent, switching back to that workspace
may result in a crash of i3. This is because while setting the urgency
of parent containers, 'parent' may become NULL in case of floating
containers. This commit checks the validity of parent.

fixes #821

Merge branch 'master' into next

Merge branch 'fix-manage-unmap'

Bugfix: properly react to windows being unmapped before we can reparent (Thanks xeen, darkraven)

We need to verify that setting the event mask works, and we need to
include StructureNotify to get unmap events at any point in time.
Thanks darkraven for the pointer.

fixes #718

Bugfix: Correctly clear the urgency hint when the window is underneath a split-con (+test)

Previously, when you had an urgent container in a stack on some
invisible workspace (say urxvt) and you switched to it, the urgency hint
was not properly cleared.

Merge branch 'fix-assign-render'

Merge branch 'master' into next

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