Prevent moving out of fullscreen containers.

Restrict directional focus in fullscreen.

This reuses the same fullscreen focus logic to ensure that focus
doesn't escape a fullscreen container.

Allow focus child/parent when in fullscreen.

This is now restricted according to the already defined fullscreen
focus constraints. Test case 157 was removed, as we don't prevent
level up/down in fullscreen anymore. Those commands are properly
tested in fullscreen by test case 156.

Fixes: #612

Refined the fullscreen focus constraints.

Basically, a focus change can't escape a fullscreen container. The
only exception is per-output fullscreen containers, as you should
be able to focus a container in a different workspace in this case.

This is an improvement on 4eab046e, now considering the difference
between global and per-output fullscreen and taking the tree
structure into account to determine what escaping the fullscreen
container means. It only affects targeted focus commands in the
form "for_window [...] focus", but it lays the foundation for
forthcoming fixes to all other focus commands.

Merge branch 'master' into next

Merge branch 'use-xkb-kc2ks'

Replace deprecated XKeycodeToKeysym() with XkbKeycodeToKeysym()

Merge branch 'master' into next

Merge branch 'tests-focus-parent-child'

Get rid of references to level up/down.

In v4 syntax, that's focus parent/child.

Merge branch 'master' into next

Merge branch 'fix-keycodes'

Fix errors in i3.config.keycodes (bindsym -> bindcode)

Merge branch 'master' into next

Merge branch 'fix-stray-ws-number'

Fix stray workspaces "number 1" being created when having workspace number 1 in your config

Merge branch 'master' into next

Merge branch 'i3bar-fix-null'

i3bar: Don’t crash when full_text is missing or null in the JSON input (Thanks fernandotcl)

Merge branch 'master' into next

Merge branch 'focus-in-fullscreen'

Allow focus w/ target when in fs in some cases.

If the target is in a different workspace, there's no reason why
we wouldn't allow the user to focus it. We already allow this when
focusing a workspace, for example.

Merge branch 'master' into next

Merge branch 'fix-workspace-number'

maybe_back_and_forth now sets render_tree

make maybe_back_and_forth static

bugfix: less differentiation between named and numbered workspaces

calling workspace by number now also checks for switching back and forth
and creates a new workspace if no workspace starting with that number is
found

also removed the obsolete tree_render() in favor of setting
cmd_output->needs_tree_render to true

tests: add test to verify workspace_auto_back_and_forth works with 'workspace number'

tests: Check that 'workspace number <number>' opens a new workspace

Merge branch 'master' into next

don’t wrap when changing workspaces by mouse wheel scrolling

Merge branch 'master' into next

Merge branch 'fix-static-analysis'

remove dead code

This was a left-over of commit 9d68d780e2ca45e231a3cb76063860c561e0453c

remove unused variable

Remove set but never read variable

This was due to the changes in commit d25e77ce7597bf0736c610a04185c8c27694c890

Merge branch 'master' into next

Merge branch 'fix-dump-log-errmsg'

Improve the main error message of i3-dump-log

Move is_debug_build() to libi3/

Fix floating precision bug

When calculating coordinates we should multiply at first otherwise
we lose precision when i3 is compiled without sse2 support.

The following code prints "Res1: 348 Res2: 349" when compiled with
-O0 -mno-sse2 and "Res1: 349 Res2: 349" with -O0 -msee2.

Note that -msse2 is default flag on 64bit OSes.

int main() {
  double a = 349.0 / 768;
  double b = 349.0 * 768;
  int res1 = a * 768;
  int res2 = b / 768;
  printf("Res1: %d Res2: %d\n", res1, res2);
  return 0;
}

Thanks guys for helping me to hunt down this one.

t/141-resize.t: do not compare float values directly. use i3test.pm::cmp_float which compares using precision of 1e-6.

Merge branch 'fix-invalid-cmd-reply'

Send proper error messages upon parser failures, use yajl for generating command replies

Fixes: #693

Merge branch 'fix-spelling'

Fix spelling of success

Fixes: #695

Merge branch 'build-mac-os-x'

log.c: len is always positive, remove useless check (Thanks Marcus)

don’t link against librt on Mac OS X (Thanks Marcus)

Make log.c figure out the physical amount of memory on Mac OS X (Thanks Marcus)

makefile: use LDFLAGS when linking test.commands_parser (Thanks Marcus)

Merge branch 'release-4.2'

debian: install the RELEASE-NOTES as upstream changelog for now

debian: bump changelog

/usr/share/applications/i3.desktop: drop the encoding key, UTF-8 is now required

debian: add i3-dump-log.1 to manpages

add release notes for v4.2

i3bar: don’t immediately redraw bars after kicking tray clients

a few little corrections for the previous commit

man/i3-input.man: Add description of options and improve examples.

i3bar: when kicking tray clients, remove them immediately

This should fix empty spaces showing up instead of tray icons sometimes.

i3bar: when tray_output == primary and there is no primary output, fall back to the first available output

i3bar: kick tray clients after output configuration changed

This makes i3bar reflect xrandr --output foo --primary changes immediately.

i3bar: Bugfix: Properly update the primary flag

Merge branch 'master' into next

Merge branch 'fix-i3bar-dying-tray'

i3bar: Bugfix: Properly reparent tray clients before killing the bar window when outputs disappear

Fixes: #655

bump copyright in --version (Thanks joepd)

cfgparse: accept force-xinerama as a synonym of force_xinerama

ipc: update the links to the python libraries on github (Thanks BusMaster)

add missing docs/refcard_style.css to git

fix broken link in docs/debugging (Thanks nh2)

debian: update changelog with new upload, add .menu file, update email, bump standards-version, fix section

replace the refcard with an HTML version (by SardemFF7)

This eliminates our build-depedency on latex.

tests: warp the pointer to in multi-monitor tests to avoid flaky tests

t/166-assign: wait until the window is definitely mapped

t/166-assign: remove unnecessary sleeps

Implement fake-outputs option (cmdline, cfg) for multi-monitor testing

This kills the dependency on xdmx and makes the testsuite simpler
and more flexible (in the output sizes / configurations).

Only resize when the left/right mouse button is used, not when scrolling (Thanks eeemsi)

Fix warning: properly initialize 'workspace' variable

Implement 'rename workspace <old_name> to <new_name>'

Implement 'move [container|window] to workspace number <number>'

Implement 'workspace number <number>' to switch to named workspaces

Prevent changing focus outside a container when scrolling on the decorations

Fixes: #557

Replace the old fullscreen container when requesting fullscreen

Fixes: #510

Implement resize <grow|shrink> <width|height>, use it in the default config

Fixes: #576

Bugfix: Properly terminate lines not ending with a newline (Thanks xeen)

Previously, we didn’t check for a newline and thus could be corrupting
formerly valid UTF-8 input, such as
    echo -n '↓'

Fixes: #671

Force WM_TAKE_FOCUS event to be sent when clicking

This fixes more problems with Eclipse.
See also commit 1962c30dfb3baee0fb98589aca575ae54d1142e0

Merge branch 'fix-eclipse-focus'

Force WM_TAKE_FOCUS event to be sent when clicking

This fixes more problems with Eclipse.
See also commit 1962c30dfb3baee0fb98589aca575ae54d1142e0

add a testcase for the previous commit

Bug 676: if scratchpad window is active on a non-visible workspace, then "scratchpad show" should move it to the current visible workspace.

Added option to select primary display on tray_output

Use (void) instead of () for functions without args (Thanks fernandotcl)

See also:
http://article.gmane.org/gmane.linux.kernel/1268792

The C compiler will handle (void) as "no arguments" and () as "variadic
function" (equivalent to (...)) which might lead to subtle errors, such
as the one which was fixed with commit 0ea64ae4.

i3bar: kill child processes when exit()ing (they might be stopped) (Thanks darkraven)

testsuite: explicitly start Xdummy with -nolisten tcp

A few style fixes for the previous commit

Use 32bit visual only when needed. Thus we could drop the --enable-32bit-visual option.

Don't multiply by sizeof(char) twice.

This is a no-op, since sizeof(char) is 1. But still, we shouldn't
multiply twice, it's misleading.

Document text_len in the draw_text description.