Merge branch 'master' into next

Merge branch 'fix-mode-switch'

Fix bindings using Mode_switch

fixes #1518

Merge branch 'master' into next

Merge branch 'revert-iconic'

Revert "Handle WM_CHANGE_STATE requests for iconic state"

This reverts commit 136b3e345bfde637d3270eafaa2d8aa3555cc3d5.

fixes #1516

Merge pull request #1515 from Airblader/feature-commands-error-messages

Improve error messages on failing commands

Improve error messages on failing commands

Fix keyboard layout change detection

…by listening to XKB_NEW_KEYBOARD events and re-allocating the keysym
map on both XKB_NEW_KEYBOARD and XKB_MAP_NOTIFY.

fixes #1302

Merge branch 'fix-keyboard-layout'

Fix keyboard layout change detection

…by listening to XKB_NEW_KEYBOARD events and re-allocating the keysym
map on both XKB_NEW_KEYBOARD and XKB_MAP_NOTIFY.

fixes #1302

Merge pull request #1503 from Airblader/defect-layout-globbing

Glob filepath when calling append_layout

Merge branch 'Airblader-defect-1480' into next

clang-format

Merge branch 'defect-1480' of https://github.com/Airblader/i3-original into Airblader-defect-1480

run clang-format (3.5.0)

Not quite sure why there are so many differences. Perhaps we’ve gotten
out of the habit of running clang-format after every change.

I guess it’d be best to have a travis hook that runs clang-format for us
and reports any problems on pull requests.

Glob filepath when calling append_layout
fixes #1500

Merge branch 'master' into next

Merge branch 'ton-master'

Merge branch 'master' of https://github.com/ton/i3 into ton-master

Merge pull request #1478 from acrisci/feature/use-default-monospace

Config: use default system monospace

Merge branch 'stevenjm-big-endian'

Merge branch 'big-endian' of https://github.com/stevenjm/i3 into stevenjm-big-endian

Fix key bindings on big-endian platforms

input_code is a uint16_t, but xcb_keycode_t is uint8_t, meaning that
only the first byte of input_code is inspected by memmem. On
little-endian platforms, this code would have worked by accident, since
the first byte of input_code represents the 8 least significant bits.
However, on big-endian platforms the first byte is the 8 most
significant bits, which means memmem is scanning bind->translated_to
for the wrong keycode (probably 0).

In order to work correctly on big-endian and little-endian platforms,
simply typecast input_code to an xcb_keycode_t and pass that to memmem.

The observed behaviour associated with this bug is that key bindings
don't work at all. This patch has been tested on an iBook G4 running
OpenBSD -current, and key bindings work properly with this fix applied.

debian: update changelog

Merge branch 'next'

Merge branch 'release-4.9' into next

release i3 v4.9

Fix incorrect y-offset for text in i3bar

When using Pango to draw text in i3bar, the y-offset of the text is
incorrectly calculated in case all characters in the string to draw are
smaller than the cached font height. Fixes #1494.

Merge pull request #1486 from acrisci/feature/docs-xvfb-tests

Docs: add hint to use xvfb-run

Config: use default system monospace

Use the default monospace font for the system in the default config.
This should be a bit more portable for systems that do not have the
recommended font installed.

Docs: add hint to use xvfb-run

Document that Xvfb can be used to run tests without an X server, and be
used to significantly speed up tests on machines with slow video cards.

Merge pull request #1471 from acrisci/feature/i3bar-use-pango-markup

i3bar: use Pango markup

free block->instance when cleaning up statusline

removed forgotten declaration of unused variable

Introduce a statusline buffer.

A buffer is introduced for the statusline which will only be copied to the actual statusline
once an entire statusline is parsed. This avoids a race condition where incompletely parsed
statuslines were rendered, causing only some status blocks to be rendered which is visible to
the user as a flickering.

fixes #1480

remove unused statusline_buffer

debian: add changelog entry for 4.8-2

i3bar: use Pango markup

Parse text within workspace buttons and the i3bar statusline as Pango
markup. This lets people specify things like font weight, text color,
background color, font size, and font family in the text of i3bar.

fixes #1468

CONTRIBUTING: document people should use the `next` branch

Properly invalidate rendering cache when updating orientation (Thanks hercek)

fixes #1445

Merge branch 'master' into next

Merge pull request #1465 from Airblader/bugfix-1430

Make click events on status blocks work if 'workspace_buttons no' is set...

Make click events on status blocks work if 'workspace_buttons no' is set.

1. Always subscribe to click events for i3bar.
2. Exit the click event handler if no current workspace was found only after clicks on status blocks have been handled.

fixes #1430

Remove one level of indentation #1465

Merge pull request #1462 from carrotIndustries/netwm_moveresize

Support _NET_WM_MOVERESIZE

Merge pull request #1460 from Airblader/bugfix-1458

Correctly calculate clicks on i3bar status blocks

Merge pull request #1451 from acrisci/feature/handle-button-release

Handle button release events

add a CONTRIBUTING file

See https://github.com/blog/1184-contributing-guidelines

Merge pull request #1453 from acrisci/feature/bar-update-font-reload

Update bar font config on reload

Support _NET_WM_MOVERESIZE

Add support for the _NET_WM_MOVERESIZE client message. This message
enables clients to initiate window moving or resizing. Toolkits like
Gtk3 use this message when the user drags a client-side decorated window
by its title bar. When Gtk detects that the window manager does not
support this client message, it uses a slow fallback implementation.

fixes #1432

Handle button release events

This enables the --release switch on mouse button bindings.

libi3: free previous font on font load

When loading a new font with `load_font`, free the previously loaded
font with `free_font`.

If no font is loaded, `free_font` will simply return (instead of
crashing because of a double free).

Update bar font config on reload

Dynamically update the font when the `reload` command is called by
reloading the font with `xcb_init_late` and adjusting the size of the
bar by updating the output dimensions with a call to the ipc.

Correctly calculate clicks on i3bar status blocks

fixes #1458

docs/debugging: use logs.i3wm.org

cking-howto: update links

Merge branch 'placeholder'

Merge branch 'master' into next

Bugfix: actually parse client.placeholder (Thanks Ingo)

fixes #1400

Merge branch 'fix-ws-con'

Merge branch 'master' into next

Bugfix: correctly restore workspaces regardless of where “type” is (Thanks dsargrad)

fixes #1395

Add mouse binding pointer position configuration

Add the `--whole-window` switch for mouse bindings. This switch controls
what part of the container the pointer must be over to trigger a mouse
binding. The default is to only trigger mouse bindings over the
titlebars. With this switch, a mouse binding will be triggered over the
main part of the window as well.

This is a breaking change to the previous behavior, which would trigger
a mouse binding with a modifier over any part of the window.

fixes #1429

i3bar: suspend the child when bars are fully obscured

Add 'mark' as allowed key to i3-save-tree output

Add lxterminal to i3-sensible-terminal

fixes #1436

Revert "don’t fix coordinates/change focus/warp when source_output was scratchpad"

This reverts commit 94808019277991a404a0a39e2a27ca1574072f3c.

This causes focus issues.

don’t fix coordinates/change focus/warp when source_output was scratchpad

Inset the urgent background of a status block for consistency with workspace buttons.

fixes #1423

Merge branch 'master' into next

Conflicts:
src/render.c

Merge branch 'fix-popup-during-fullscreen'

Bugfix: render floating windows during global fullscreen (Thanks dmurph, Mats)

fixes #1393

Fix start_application() doc about which shell is used

Since this commit:

f691a55923850a4d315450925fc98733d07b69c9

the shell that is used is the system's bourne shell (/bin/sh) and the
env variable SHELL is not considered.

No logic changes.

Include workspace con in workspace event

Send the affected workspace in the "current" property for each workspace
event for any type of workspace event that affects a particular
workspace.

fixes #1411

docs/ipc: use an actual event type (thanks Adaephon)

Extend the fullscreen command

Rather than just toggling the fullscreen modes, allow to set them
directly with:

    fullscreen enable|toggle [global]
    fullscreen disable

For compatibility, retain the previous command and its toggling behavior:

    fullscreen [global]

fixes #1120

Prevent workspace change during global fullscreen

While in global fullscreen, the workspace could be changed leaving the
fullscreen container still visible on top but losing its focus.

i3bar: Respect the urgency flag on status blocks

…by drawing urgent blocks with the same settings as an urgent workspace.

(Also use logical_px() and bar_height.)

fixes #682

Improve error message when a full_text property is missing

ensure align = left is the default and update documentation accordingly

fixes #1403

add deco_rect property to con in ipc response

Added termite to default terminals

Revert "Bugfix: Set input focus with last timestamp"

This reverts commit 9cee8dac5e9057bf2cb72edc3cf398fdc08b8d73.

fixes #1383

Bugfix: check if values are non-NULL before copying (Thanks xeen)

fixes #1397

Handle _NET_CLOSE_WINDOW client message requests

> Pagers wanting to close a window MUST send a _NET_CLOSE_WINDOW client
> message request to the root window.

We interpret this message as a request to close the con for the given
window.

See: http://standards.freedesktop.org/wm-spec/wm-spec-latest.html#idm140200472668896

fixes #1396

bugfix: copy binding before run

Copy the binding struct before running it and use this copy to emit the
binding event.

This fixes a crash when the command `reload` is used in a binding when
the binding event is emitted.

merge the formerly debian-specific x-terminal-emulator patch

I think instead of maintaining distribution-specific patches forever,
it’s a bit easier to have them applied upstream, especially since they
don’t break anything.

fixes #876

set DesktopNames (which gdm uses) (Thanks Yves-Alexis)

Use command -v (built-in) instead of which(1) (Thanks val)

fixes #1380

t/234-layout-restore-output: Remove unused import

List::MoreUtils is imported but not used.

Makefile.PL: Add Inline::C dependency

Inline::C is no longer shipped with Inline.

Testcases: rename StartXDummy to StartXServer

Rename the package StartXDummy to StartXServer in the testcases library
because XDummy is no longer used.

No logic changes.

Bugfix: check symbol for NULL in binding json dump

When dumping a binding, as is done during the binding event, check
symbol for NULL. If it is, dump json null. This prevents a crash when
running a binding that was configured with bindcode.

fixes #1379

check for AnyEvent::I3 ≥ 0.16 in the binding event test

Implement the ipc 'binding' event

The binding event will be triggered when a binding is run as a result of
some a user action. The binding event has the following properties:

change: (str) Currently this will only be "run" but may be expanded in
the future. Included for consistency with other events.

binding: (map) the serialized binding

The "binding" member will have these properties:

input_type: (str) either "keyboard" or "mouse"

input_code: (int) the xcb keycode of the keyboard binding if it was
provided or the mouse button if it is a mouse binding.

symbol: (str) the string representation of the input code

command: (str) the bound command

mods: (list of str) a list of the modifiers that were pressed as string
symbols

fixes #1210

t/114-client-leader: fix race by waiting until the window is mapped

Testcases: Use Xephyr instead of XDummy

Replace the XDummy script with Xephyr. This is done because of some
changes in the Xorg server that make XDummy difficult to use.

Rename library internal variables and function names to replace "xdummy"
with "xserver" to show this change (except for renaming the package and
lib file for better git history).

Rename the switch `--keep-xdummy-output` to `--keep-xserver-output`.
This switch should now be rarely used because Xephyr requires less set
up.

Replace "xdummy" with "xephyr" in comments and utility help
information. Update docs to show the new dependency.

fixes #1367

Bugfix: use the command parser to properly extract workspace names

fixes #1377

Don't draw borders wider than actual width

Rectangles passed to function xcb_poly_fill_rectangle are of type
xcb_rectangle_t and defined as:

    struct xcb_rectangle_t {
        int16_t  x;
        int16_t  y;
        uint16_t width;
        uint16_t height;
    }

The rectangles for the right and lower border had a width and height,
respectively, greater than the actual border width.

Furthermore, offset the bottom border to not overlap with the right one
and, for the top border, use r->width instead of con->rect.width as with
the other borders.