t/35-floating-focus: rewrite testcase to use windows instead of empty cons

Bugfix: set WM_STATE to WITHDRAWN when an app unmaps their window(s) (+test)

Fixes: #362

Bugfix: Fix the WANT_QSTRING state

Bugfix: Fix focus follows mouse for non-default layout cons (Thanks phnom)

Fixes: #361

s/seperate/separate (Thanks Jon)

Bugfix: Ignore focus when attaching cons while restoring the layout

Fixes: #369

Bugfix: Correct string/quoted string parsing for the commands  exec, workspace, nop, restore and mark (Thanks SardemFF7)

Fixes: #380

Bugfix: Don’t append the --restart parameter on each restart (Thanks aniou)

Fixes: #384

Bugfix: Don’t crash when dock clients set the urgency hint (+testcase)

Bugfix: Don’t fill split cons etc. with client background color (fixes nested decoration rendering)

Fixes: #359

Bugfix: Restore border_style when restarting inplace (Thanks aniou)

Fixes #385.

ipc: change border_style to human-readable string instead of enum value

Bugfix: Don’t run into an endless loop when killing con with children (Thanks mseed)

When a tabbed container had more than one child and at least the first one
supported WM_DELETE, i3 entered an endless loop when killing that tabbed
container. This was due to tree_close only sending WM_DELETE without actually
removing the child, while the loop in tree_close assumed that with every call
of tree_close one child would be removed.

add coverage target to makefile to generate a coverage report

tests: add --coverage-testing option to complete-run.pl

makefile: add COVERAGE flag

Bugfix: assign BORDER_BOTTOM instead of BORDER_RIGHT (Thanks Jan)

cfgparse.y: define types (Thanks Merovius)

re-indent cfgparse.y

cmdparse.y: set type on the tokens/non-terminals (Thanks Merovius)

re-implement assigning windows to workspaces

Bugfix: Before rendering, attach the con to its floating_con

Otherwise, the rendering will produce negative coordinates.

Bugfix: When re-assigning floating windows to a different output, use the last focused workspace, not the first

Bugfix: Fix the client background rectangle calculation (Thanks Mike)

Really fixes #364.

Bugfix: For fullscreen cons, use a deco_height of 0 to correctly render the background color

This should fix #364.

Bugfix: Correctly render decorations in tabbed containers (don’t overlap)

This fixes a regression introduced in b644fb5f26c1768b70c5b49d8cd917a63a2d1d91.

wizard: actually write the output config

wizard: check if the config file does not already exist and if we can create it

initial commit of the i3-config-wizard (GUI working, functionality incomplete)

Bugfix: Add missing tree_render() when handling the urgency hint (Thanks mxf)

Fix unaligned memory access on sparc (Thanks David Coppa)

remove useless checks, size_t != ssize_t :)

Make code compatible with yajl 2.0 *and* 1.0

Bugfix: Ensure that all outputs have a ->con before handling disabled outputs (Thanks JimdiGriz)

Steps to reproduce:
• xrandr --output VGA1 --auto
• xrandr --output LVDS1 --off

fix warning by removing 'const' from ipc_socket_path

tests: make complete-run.pl scan tests for !NO_I3_INSTANCE!, simplify code

tests: add testcase for the different socket path locations

Use XDG_RUNTIME_DIR when available

XDG_RUNTIME_DIR is the volatile runtime data dir provided by modern
session manager such as systemd

Bugfix: Ignore EnterNotifies generated by UnmapNotifies

Actually, commit 1c5adc6c35cffaedc08c7d1dd1b03a3269d1367c commented out code
without ever fixing it. I think this was responsible for the 'workspace
switching sometimes does not work' bug. My observations:

Had it again today and analyzed a log of it. Looks like after unmapping the
windows on one workspace (in my case: chromium, eclipse, urxvt, focus on
eclipse) we get UnmapNotify events for chromium and eclipse, but then we get an
EnterNotify for the terminal (due to unmapping the other windows and therefore
mapping the terminal under the cursor), only afterwards the UnmapNotify
follows.

So, there are two things wrong with that:

• We handle EnterNotifys for unmapped windows

• Unmapping windows sometimes works in a sequence, sometimes the sequence gets
  split. Not sure why (if unmapping can take longer for some windows or if our
  syncing is wrong -- but i checked the latter briefly and it looks correct).
  Maybe GrabServer helps?

• We don’t ignore EnterNotify events caused by UnmapNotifies. We used to, but
  then there was a different problem and we decided to solve the EnterNotify
  problem in another way, which actually never happened (commit
  1c5adc6c35cffaedc08c7d1dd1b03a3269d1367c).

Rename bind to bindcode

Also fallback when using just 'bind' to be backward-compatible

x: Set pixmap as background window, saves a lot of CopyAreas

Bugfix: Center floating windows with invalid coordinates on current ws

Bugfix: Check if a floating window’s coordinates are within a different workspace when managing (Thanks Merovius)

Fixes: #297

Bugfix: Assign floating cons to correct workspace when moving between monitors (Thanks dothebart)

Fixes: #371

Bugfix: When moving floating cons to other workspaces, attach them to the workspace

Bugfix: Don’t set the layout when it’s default layout anyways

Fixes some nasty side-effects

Rename new_container to workspace_layout

Implemented config key 'new_container'

optimization: when moving floating windows, render/push only the floatingcon

Bugfix: Flush the Xlib connection after creating cursors

Fixes a race condition where the cursors were created after we were already
using them.

x: skip creating pixmaps when the rect is actually invisible

This fixes a few X11 errors.

bugfix: fix race condition where window titles were not correctly updated

Fixes: #351

x: skip x_draw_decoration when con is not mapped

This commit makes workspace switching completely free of cache misses, so
decorations are not re-rendered when switching workspaces.

x: recurse x_push_node in focus order. reduces flickering when switching workspaces

Skip FocusIn events with mode == NOTIFY_MODE_GRAB or NOTIFY_MODE_UNGRAB

According to the Xlib Programming Manual section 10.7.2 [1], these events are
generated when keyboard grabs activate/deactivate, while we are only interested
in focus changes which are done by other programs independend from the
keyboard.

[1] http://tronche.com/gui/x/xlib/events/input-focus/grab.html

little coding style fixes, fix compilation warning

If the socket path isn't specified, write it to /tmp.

FREE() already nulls the pointer

Set the I3_SOCKET_PATH atom to the expanded path.

Focus cons when moving to a visible workspace on a different output (Thanks mseed)

Fixes: #355

Bugfix: Fix switching workspaces on multi-monitor setups (Thanks mseed)

Fixes #356

Bugfix: Also invalidate caches of the following cons in a split con on cache miss (Thanks fernandotcl)

Bugfix: Also update pixmaps when the position of the deco_rect has changed (Thanks fernandotcl)

refactor handlers.{c,h}: declare the handlers static, remove unnecessary parameters

remove unused struct Colorpixel

Handle FocusIn events generated by clients and update decoration accordingly (Thanks mseed)

Remove debugging syncs

Re-implement rendering to pixmaps (double-buffering) and caching decorations

Bugfix: Actually re-attach dock clients when outputs get disabled (Thanks phnom)

Fixes: #348

Bugfix: Don’t use the workspace where focus is for deleting workspaces when switching (Thanks mseed)

Fixes: #353

Bugfix: also invalidate focused_id when the to_focus window is not mapped

This fixes a bug where focus might not be set correctly when changing workspaces

Handle saved_configpath in get_config_path, fix memleak in current_configpath handling, update atoms after reloading (Thanks fernandotcl)

Bugfix: correctly store the config path for using it for I3_CONFIG_PATH later

Bugfix: fix null-pointer dereference when IPC is disabled (Thanks Merovius)

i3-msg, i3-input: get the I3_SOCKET_PATH atoms if socket path was not specified

Set the I3_SOCKET_PATH and I3_CONFIG_PATH atoms on the X11 root window

Bugfix: When there’s nothing to focus, focus the root window (Thanks fernandotcl, ThePub)

tests: add testcase for WM_TAKE_FOCUS

requires the very latest checkout of X11::XCB

rendering: correctly draw background rect (Thanks phnom)

Fixes #347

remove hard-coded paths since we now use pkg-config for all the dependencies

remove obsolete xcb-property line (Thanks SardemFF7)

i3-input: remove dependency on xcb-event

remove obsolete comment

Only send WM_TAKE_FOCUS when the client supports it in the protocols atom

Fixes opening xterm, for example

Make i3 compatible with the very latest xcb

This involves:
 • Compiling with xcb-util instead of xcb-{atom,aux} (they merged the libraries)
 • Not using xcb-{event,property} anymore (code removed upstream)
 • Not using the predefined WINDOW, CARDINEL, … atoms (removed upstream)
 • Using the new xcb_icccm_* data types/functions instead of just xcb_*
   (for example xcb_icccm_get_wm_hints instead of xcb_get_wm_hints)

Also I refactored the atoms to use x-macros.

Send WM_TAKE_FOCUS to clients when setting focus (fixes java swing problems)

remove orientation and fall back to default behaviour

change the config parser to use default_orientation instead of new_container_orientation

It’s a shorter and probably more meaningful description as it is not
immediately clear what a container exactly is when first installing i3.

cleanup code of workspace rotation on output changes

Added config key for default orientation of containers (new_container_orientation) and added support in randr.c for automatically changing the orientation when user does a xrandr rotate.

When making floating cons tiling, re-insert next to the next focused *tiling* con (Thanks mseed)

Fixes: #337 and #350

fix small warnings when compiling with DEBUG=0

cmdparse: correctly parse con_id/id (fixes warning)

cmdparse: expect 4 shift/reduce conflicts

cmdparse: don’t allow empty operations

cmdparse: don’t allow empty commands

remove usless 'operation' token, already handled by 'operation optwhitespace'

remove unused current_bindings (left-over from cfgparse.y)

re-implement support for MappingNotifys

ewmh: bump copyright