Merge pull request #1737 from Airblader/feature-xdotool-on-travis

Install xdotool to run tests requiring it on Travis.

Install xdotool to run tests requiring it on Travis.

Merge pull request #1735 from shdown/strdup

i3bar: fix freeing static strings

i3bar: fix freeing static strings

name and color fields of blocks are freed in clear_statusline, so they
need to be strdup'ed.

Merge pull request #1733 from Airblader/feature-1732

Added a separate color directive for the binding mode indicator

Merge pull request #1734 from Airblader/bug-1538-1

Fix warning about potentially uninitialized variable.

Fix warning about potentially uninitialized variable.

Use the introduced binding_mode_* colors to draw the binding mode.

fixes #1732

Added the 'binding_mode' color directive to the documentation.

Introduce a config directive 'binding_mode' in the 'bar' config, pass it through the IPC and parse it in i3bar.

Merge pull request #1725 from Airblader/bug-1056

Allow window decoration to change focus

Use XCB_NONE instead of hard-coded 0.

Allow hovering over the window decoration to change the focus if the parent container is split horizontally or vertically.

fixes #1056

Added a testcases for #1056 to test whether hovering over a window decoration causes the corresponding window to be
focused.

Merge pull request #1722 from Airblader/bug-1701

Grab all buttons when managing a window

Improve logging of button events.

Grab all buttons when managing a window to also allow 'bindsym --whole-window button4 …' to work correctly.

fixes #1701

docs/ipc: add veelenga/i3ipc-ruby

Merge pull request #1714 from Airblader/feature-docs-6

Fixed syntax for tray_output in userguide

Merge pull request #1716 from botovq/next

mkdirp: do not throw an error if directory exists

mkdirp: do not throw an error if directory exists

If I restart i3 4.10.2 twice, e.g. with

$ i3-msg restart; sleep 3; i3-msg restart

the second time I get the following two errors:

05/22/15 10:46:03 - ERROR: mkdir(/tmp/i3-theo.toAK7N) failed: File exists
05/22/15 10:46:03 - ERROR: Could not create "/tmp/i3-theo.toAK7N" for storing the restart layout, layout will be lost.

The first one is from mkdirp() in src/ipc.c and the second one is from
store_restart_layout() in src/util.c.

Notice that I do _not_ get the ``open()'' or ``Could not write restart layout to
...'' error messages, so the layout writing code after line 260 in
store_restart_layout() succeeded and the layout isn't actually lost.  Thus,
these error messages are a bit misleading, especially the second one (which is
triggered by the failure of mkdirp()).

POSIX says about `mkdir -p':

``Each dir operand that names an existing directory shall be ignored without
error.''

Therefore, I suggest the following simple patch that makes mkdirp() succeed if
the named file exists and actually is a directory.  This silences the second
error as well.

Fixed syntax for tray_output in userguide.

Merge pull request #1706 from Airblader/feature-docs-5

Made syntax of syntax descriptions consistent:

Made syntax of syntax descriptions consistent:

* <xyz> denotes that some string must be used which is not a fixed value (e.g., a command), but a variable string (text, a number, ...)
* [xyz] denotes that the parameter is optional
* abc|xyz denotes that either abc or xyz must be given

Merge pull request #1698 from Airblader/feature-1696

Added 'move position mouse'

Protect "move position mouse" against a NULL access.
This could happen if two outputs are set up to have a gap in between them and the mouse cursor
being in said gap when the command is triggered.

Added testcases for 'move position mouse'

Merge pull request #1704 from acrisci/bug/cmd-detect-base-16

Detect base 16 in cmd criteria

Added 'move position mouse' command to the userguide.

Implemented new command 'move [window|container] [to] position mouse|cursor|pointer

fixes #1696

Detect base 16 in cmd criteria

Try to detect base 16 numbers given to `con_id` and `id` for command criteria
by setting the base of strtol to 0. This should also detect octal.

This is necessary because mouse bindings now may be serialized as hex as of
5c32de4.

Merge pull request #1700 from mh21/tray-icon-size-cleanup

Tray icon size cleanup.

Merge pull request #1693 from mh21/wm-class-garbage-no-copy

Don't duplicate property value on class change.

No memcpy on class change.

Introduce sstrndup wrapper.

Merge pull request #1638 from hwangcc23/fix-1489

Add a timeout: delay_exit_on_zero_displays

userguide: quoted strings need to be used, escaping isn’t possible

fixes #1678

Merge pull request #1689 from Airblader/feature-docs-4

Debugging docs – minimal setup, small log file, closed source software

Added a section to the debugging docs:
* Motivate users to come up with clear and minimal instructions on how to reproduce a problem before submitting an issue.
* Encourage users to restart i3 before reproducing the problem so that the log file can stay small and noise-free.
* Mention the non-support of closed source software.

Merge pull request #1685 from Deiz/fix-new-workspace-float

Initialize workspace rect to the output's upon creation

Tray icon size cleanup.

- icon_size instead of font.height
- consistent spacing with tray_spacing_px

Initialize workspace rect to the output's upon creation

The comment immediately following implied that this was the intended
behaviour. Not doing so means that compound commands that both move a
window to a new workspace as well as do something that depends on the
workspace's geometry (e.g. 'move position center' or 'floating enable'
on a tiled window) would use the workspace's calloc'd 0x0+0x0 geometry.

Merge pull request #1680 from acrisci/bug/randr-disabled-outputs

Check if output is disabled in handle_output()

Merge pull request #1657 from Georgiy-Tugai/fix-flickering-shortened

i3bar: fix flickering shortened status bar on other output(s)

Merge pull request #1683 from Airblader/bug-click-detection

Only detect clicks within the statusline width.

Merge pull request #1688 from Airblader/feature-1687

Added a note to the debugging docs about sensitive data.

Added a note to the debugging docs about sensitive data.

resolves #1687

Merge pull request #1684 from acrisci/bug/focus-without-input-hint

Ignore InputHint when not in WM_HINTS

Merge pull request #1672 from acrisci/bug/fix-freebsd-bindmouse

bugfix: serialize con_id with %p in run_binding()

Ignore InputHint when not in WM_HINTS

When InputHint is not in WM_HINTS (i.e., the flag is not set), treat the window
as if the InputHint was set (the default behavior). This means that i3 will
focus the window when it becomes managed.

fixes #1676

Bugfix: serialize con_id with %p in run_binding()

%p is equivalent to either %x or %lx, depending on the pointer size of the
platform. Before this commit, we always used %d, which has the same behavior
on Linux, but is not automatically expanded to %ld on e.g. FreeBSD.

fixes #1661

Only detect clicks within the statusline width.

If the user clicks on the very top of i3bar above a tray icon, i3bar might
mistakenly trigger the click event for the last status block. This is due to
the fact that the click detection considers the separator width of the block
even though on the last block this is ignored, incorrectly spanning the block's
width partially (or entirely) across the tray area.

userguide: add a note to both “exec”s about semicolon and comma

fixes #1678

Check if output is disabled in handle_output()

Check if the `connection` of the randr output is
XCB_RANDR_CONNECTION_DISONNECTED and disable the output if it is.

This fixes an issue where the output would not be disabled if the output was
physically unplugged from the machine.

Merge pull request #1664 from Airblader/feature-wm-state-hidden-2

Set _NET_WM_STATE_HIDDEN in tabbed/stacked containers

Added testcases for setting _NET_WM_STATE_HIDDEN on unfocused containers in tabbed/stacked containers.

Set the _NET_WM_STATE_HIDDEN atom on windows that are currently not visible due to being in the non-focused tab of a stacked or tabbed container.

fixes #1648

Added 'con_is_hidden' to check whether a given container is visible to the user assuming its workspace is visible.
This is useful for determining whether we want to set the _NET_WM_STATE_HIDDEN atom on the window.

Merge pull request #1665 from Airblader/feature-1658

Added criterion 'window_type'

Merge pull request #1669 from Airblader/feature-docs-2

Documentation Improvement – Debug i3bar

Merge pull request #1636 from Deiz/update-notices

Update copyright notices and get rid of ranges

Merge pull request #1666 from Xarthisius/undef_FALSE

Fix ‘FALSE’ undeclared (first use in this function) when i3 is built w/o PANGO

i3bar: fix flickering shortened status bar on other output(s)

Fix ‘FALSE’ undeclared (first use in this function) when i3 is built without PANGO support

Update copyright notices and get rid of ranges

The script used to make these changes can be found at:

   https://gist.github.com/Deiz/32322020f76d23e2bf8f

Merge pull request #1674 from Airblader/feature-docs-3

Adapt release.sh to update the latest version in the debugging docs.

Cleanup: delete comments that were accidentally copy-pasted and are incorrect for these test cases

Small cleanups:
* Implement criterion specs just like it is done in the spec for the config
* Declare variables in test case early so the 'my' keyword can be dropped in the actual test cases

Added testcase for 'window_type' criterion when used as a command.

Added testcase for changing _NET_WM_WINDOW_TYPE after the window is already managed.

Handle changes to _NET_WM_WINDOW_TYPE after the window has been managed.

Implement new criterion 'window_type = normal|dialog|utility|toolbar|splash|menu|dropdown_menu|popup_menu|tooltip'

fixes #1658

Provide instructions on how to debug i3bar

Adapt release.sh to update the latest version in the debugging docs.

contributing: mention that we cannot support closed-source software

Merge pull request #1662 from Airblader/feature-move-to-mark

Allow moving windows to marks

Added test case for transfer of urgency flag when moving a window to a mark.

fix commands parser test

Added testcase for moving a window to a workspace holding the mark

Disallow moving a container to itself since this would crash i3.

Added testcases for 'move window|container to mark <str>'

fixes #1643

Implement the command 'move container to mark <mark>' using 'con_move_to_mark'.

Added 'con_move_to_mark' to move a container to the container holding a certain mark.

Extract a function from con_move_to_workpsace that deals with moving a container to some other container rather than being restricted to moving to a specific workspace.

added function 'con_by_mark' to look up a con holding the given mark

Added configuration directive for 'move [container|window] [to] mark <str>'

Merge pull request #1632 from Deiz/binding-border

Add a --border flag to enable mouse binds to trigger on border click

Added testcase for criterion 'window_type'.

Merge pull request #1651 from tanderson92/pkgconfig

Respect EXEC_PREFIX and a users' choice of PKG_CONFIG.

release.sh: add missing -t parameter for sendmail

release.sh: add missing commit step for debian changelog

release.sh: fix encoding and concatenating changelog

release.sh: set up master branch for pushing the website

release.sh: checkout correct branch before copying docs

release.sh: 4.10.2 release

debian: update changelog

Merge branch 'master' into next

Merge branch 'release-4.10.2'

release i3 4.10.2

Respect EXEC_PREFIX and a users' choice of PKG_CONFIG.

The Makefiles should put binaries in $(EXEC_PREFIX) and
architecture-independent files in $(PREFIX). Also a user may have a
prefixed- pkg-config, as in the case of cross compiling on Exherbo
Linux, so respect the well-accepted $(PKG_CONFIG) variable for this
purpose.