Merge pull request #3657 from jeffhuxen/next

Added note which config directives could be used at runtime

Merge pull request #3643 from Iskustvo/next

Adding new IPC library(i3-ipc++) in documents.

Merge pull request #3655 from orestisf1993/workspace_output_spaces

cfg_workspace: Accept outputs with spaces again

Added to note to clarify which config directives could be used at runtime.

Added new IPC library(i3-ipc++) in documents.

cfg_workspace: Accept outputs with spaces again

This is a regression from bce088679.

An other way to fix this would be to concatenate strings inside the
strtok loop when an output starts with a double quote but I'd rather
let the parser do the word splitting.

Fixes #3646

Merge pull request #3654 from chrhasse/next

Add explicit reference to glib2 in automake

Add explicit reference to glib2 to automake

Merge pull request #3637 from i3/stapelberg-patch-1

x.c: correctly free con->frame_buffer in _x_con_kill

x.c: correctly free con->frame_buffer in _x_con_kill

This fixes a crash which I could not reproduce in a testcase with reasonable effort, but the user reported the fix works.

Compare with src/x.c:946.

fixes #3554
fixes #3645

travis: remove deprecated docker login -e flag (#3651)

Merge pull request #3640 from orestisf1993/partial-ucs2

convert_utf8_to_ucs2: Allow partial conversion

convert_utf8_to_ucs2: Allow partial conversion

Fixes #3638.

fix travis build by switching away from deprecated-2017Q3 (#3650)

Also remove “sudo: false” as per
https://blog.travis-ci.com/2018-11-19-required-linux-infrastructure-migration

Merge pull request #3614 from NilsIrl/next

Add proper return code for i3-msg

Merge pull request #3632 from orestisf1993/workspace_init

create_workspace_on_output: send workspace init event

create_workspace_on_output: send workspace init event

Fixes #3595
Like the issue mentions:
> instead of the newly created workspace (not referenced by variable
> here) the `"init"` event is fired with the current workspace (`ws`).

Plus, there was another issue where duplicate workspace init events
where being sent because of workspace_get().

304-ipc-workspace-init.t: Subtest "move workspace to output" fails with
current next.

Fixes #3631
No event was being sent here:
https://github.com/i3/i3/blob/2d6e09a66ad9b3e01588c515ae66476de8903754/src/randr.c#L487

533-randr15.t: I confirmed that SKIP still works if the xrandr command
fails.
Added test fails with current next.

Add proper return code for i3-msg

i3-dump-log: make log message a little more clear (#3618)

This came up when trying to debug an issue.

Merge pull request #3607 from vacuus/3546-fix-misleading-doc

Clear up confusion between event and reply types.

Reword documentation to make clear the difference in enumeration between event and reply types.

cmd_exit: Let i3_exit handle shutdown (#3600)

- __lsan_do_leak_check() will terminate the process, so move it to the
end of the function.
- ev_loop_destroy() must be called after ipc_shutdown() because the
latter calls ev_ functions.

Fixes #3599

Merge pull request #3590 from nejni-marji/next

Make binding modes case sensitive

closes #3587

Update userguide to describe border styles better

Describe the difference between normal and pixel titlebars better

Thanks to @JonWoodlief

Make binding modes case sensitive

Use ipc queue for all messages (#3585)

I was able to reproduce #3579 in Linux by running:
`sudo sysctl net.core.wmem_default=10000`

If a subscription message was too big to be sent at once, it was
possible to break a client by sending a reply to an other message sent
by the client. Eg:
- Write 8192 out of 11612 bytes of a workspace event.
- Blockingly write the reply to a workspace change message.
- Write the rest 3420 bytes of the workspace event.

This commit fixes this by utilizing the ipc queue for all types of
writes.

ipc_receive_message can only be called from a callback started in
ipc_new_client. This callback uses the same file descriptor with the
client also created in ipc_new_client. When the client is deleted, the
read callback is now also stopped. Thus, we can assume that whenever
ipc_receive_message is called, the corresponding client should still
exist.

- ipc_client now contains pointers to both write and read watchers. When
freed, a client will stop both of them.
- IPC_HANDLERs now work with ipc_clients instead of fds.

Fixes #3579.

Merge pull request #3584 from orestisf1993/popup-SIGSEGV

Fix crash with popups when fullscreen is non-leaf

Fix crash with popups when fullscreen is non-leaf

Introduced in b3e69ed12

Fixes #3582

Merge pull request #3574 from pclouds/no-newline-in-errx

Remove \n from errx and die messages

Remove \n from errx and die messages

errx() already appends \n internally. "\n" in the error message will
result in a blank line after the message. die() is just a wrapper around
errx() so it receives the same treatment.

Merge pull request #3563 from CyberShadow/doc-mark-todo

userguide: Un-hide a TODO block completed in 2011

Merge pull request #3568 from orestisf1993/floating_reposition-render_con

Fix: render_con shows floating containers on wrong workspace

Fix: render_con shows floating containers on wrong workspace

After 204eefc. Alternative fix:
    diff --git a/src/floating.c b/src/floating.c
    index f5c61782..6dd79668 100644
    --- a/src/floating.c
    +++ b/src/floating.c
    @@ -954,7 +954,7 @@ bool floating_reposition(Con *con, Rect newrect) {
             con->scratchpad_state = SCRATCHPAD_CHANGED;

         /* Workspace change will already result in a tree_render. */
    -    if (!reassigned) {
    +    if (!reassigned && workspace_is_visible(con_get_workspace(con))) {
             render_con(con);
             x_push_node(con);
         }
but I don't think that the extra complexity is worth it.

Change in handlers.c because of d2d6d6e0 where the bug also appears.

Fixes #3567

Merge pull request #3566 from orestisf1993/focus-events

Focus events

Invalidate last_focused when focusing the EWMH support window

Fixes #3562

Fix: killing unfocused window shouldn't produce focus event

Update tree_close_internal documentation in tree.h

After f90840337

userguide: Un-hide a TODO block completed in 2011

The userguide contained a commented-out section for marks, which
included the line:

> TODO: make i3-input replace %s

The line was added in a26a11c6099f82fc9fd8b87f8bda128408b4f7c9 (May
2011), at which point i3-input did not have a -F switch. The switch
was added only in 1737a78fcd8025e11398dbcf5acd65c6a07ae86d (September
2011), but the documentation was never updated to enable the
commented-out examples.

Merge pull request #3560 from CyberShadow/doc-mark-replace

userguide: Document mark --replace flag

userguide: Document mark --replace flag

- Explicitly document --replace, which was previously only mentioned
  in the command syntax.

- Improve wording: "a window can only have one mark" is slightly
  misleading because it appears to describe the limitation as a
  property of the model, whereas this actually pertains the mark
  command.

Merge pull request #3557 from orestisf1993/regress-float-move

attach_to_workspace: set new parent before tree_render

attach_to_workspace: set new parent before tree_render

on_remove_child calls tree_close_internal which calls tree_render and
the tree is in an invalid state if con->parent still points to the old
parent.

Fixes #3556

Merge pull request #3510 from orestisf1993/tree.t

Introduce cmp_tree test function

Introduce cmp_tree test function

Related to #3503

Merge pull request #3551 from xzfc/baf-restart

Preserve back_and_forth during restart

Preserve back_and_forth during restart

Add new key "previous_workspace_name" to the json dump of the root container.

Merge pull request #3553 from stapelberg/hidpi

userguide: add a section about hidpi displays

userguide: add a section about hidpi displays

This is a continuation of #3438.

Merge pull request #3549 from xzfc/small-fixes

Small fixes

Merge pull request #3550 from tlercher/3535-display-check

Fix #3535 - Check for DISPLAY when requesting version information

Fix #3535 - Check for DISPLAY when requesting version information

Code style: fix misaligned and misindented comments

Remove unused con_get_next()

This function has unused for a long time since the commit
8f4b9ddaa43ae61932fd702ad7178c2b4e07cfb9.

Remove unnecessary code in route_click()

This case is handled by resize_find_tiling_participants() anyway which
is introduced in the commit dbec5eb90585bc22752331f51d8a6bc90d21889c.

commands.c: Add missing error replies

Merge pull request #3547 from stapelberg/conn

Bugfix: use restore_conn, not conn

Bugfix: use restore_conn, not conn

Using the wrong X11 connection breaks the libev event handling model:
xcb_flush() must be called immediately before handing control to libev.

Before this fix:

1. xcb_prepare_cb would read and flush conn
2. restore_xcb_prepare_cb would read and flush restore_conn,
   BUT also inadvertantly call xcb_flush(conn), resulting in new
   events being filled into the XCB event queue
3. libev waits for new events
4. after 1 minute, libev times out and the events are processed

Diagnosed using strace on testcases/complete-run.pl.

related to commit 0d8b6714e39af81cbd6f4fbad500872a715dea24

related to #3510

Merge pull request #3541 from orestisf1993/title_align

Apply title_align to non-leaf containers

Apply title_align to non-leaf containers

Additionally, marks will now display for non-leaf containers.

Fixes #3540.

Merge pull request #3514 from xzfc/2742-shape

Add input and bounding shapes support (#2742)

Add input and bounding shapes support (#2742)

Basic idea: if the window has a shape, set the parent container shape as
the union of the window shape and the shape of the frame borders.

Co-authored-by: Uli Schlachter <psychon@znc.in>

Log window id in state_for_frame()

switch to clang-format-6.0 (#3533)

Merge pull request #3432 from orestisf1993/aspect-ratio

 Fix aspect ratio bugs

Merge pull request #3524 from orestisf1993/regress-randr

randr.c: Fix regression with focusing NULL container

randr.c: Fix regression with focusing NULL container

This was introduced in db3b9e41874400958cf85b461a5d1ff04a398fa3 which
removed the NULL check for next.

Fixes #3523.

Merge pull request #3521 from c-edw/feature/StripWorkspaceDocs

Update userguide docs for strip_workspace_*.

Update userguide docs for strip_workspace_*.

Re-render floating cons alone when possible

render_con: Get rid of render_fullscreen argument

Only true for the fullscreen container and doesn't affect any of its
children. Thus, we can get the same result by checking
->fullscreen_mode.

Fix aspect ratio bugs

- ICCCM says: > If a base size is not provided, the minimum size is to
be used in its place and vice versa.
i3 didn't obey the "vice versa" part. Min size and base size are both
saved without replacements in window_update_normal_hints,
floating_check_size makes the needed replacements if either was not
provided.
- Aspect ratio is now saved correctly in manage_window because
window_update_normal_hints is called.
- i3 didn't save the aspect ratio if the window conformed the given
aspect ratio range when handle_normal_hints was called. If the window
was resized to a size outside of the given bounds, i3 didn't correct it.
- Aspect ratio now affects only tiling windows, like the rest of the
normal size hints
- The aspect ratio calculation is now done without a loop

A real life example of how these changes affect the workflow:
An mpv window, when playing a video, sets its min == max aspect ratio
during mapping. i3 ignored these hints. When resized, the window's
aspect ratio was not preserved. With this commit, resizing floating mpv
windows will always preserve the aspect ratio.

floating_resize: Use uint32_t

floating_check_size: Use window variable

Merge pull request #3484 from xzfc/3476-export-i3sock

Export I3SOCK

Merge pull request #3397 from orestisf1993/randr-enable-disable

Fix bugs in enabling & disabling randr outputs

Merge pull request #3516 from orestisf1993/truncate-utf8

Truncate wm_name utf8 strings to first zero byte

Truncate wm_name utf8 strings to first zero byte

Fixes #3515

Merge pull request #3511 from aksel/247-gaps-resize-fix

For resizing, convert pixel diff to percentage, based on parent.

For resizing, convert pixel diff to percentage, based on parent.

Previously, it first calculated one of the containers' next percentage, and then subtracted the previous percentage to find the actual change.

Now it directly calculates the change, and subtracts and adds the change to the two affected containers.

Added util function con_rect_size_in_orientation.

Removed px_resize_to_percent; inlined, using con_rect_size_in_orientation.

Also, prematurely return when pixel diff is 0, as no action is necessary.

This is related to [this issue on i3-gaps](https://github.com/Airblader/i3/issues/247).

Merge pull request #2954 from orestisf1993/swap-for-floating

Rewrite con_swap to work only with queue operations

Merge pull request #3508 from orestisf1993/load_layout-marks

load_layout: Correctly mark non-leaf containers

load_layout: Correctly mark non-leaf containers

Example problematic layout:
    {
        "layout": "splith",
        "marks": ["H1"],
        "nodes": [
            {
                "swallows": [
                    {
                        "class": "^a$"
                    }
                ]
            }
        ]
    }

Since the marks were added to the json_node during end_map, the
container that ended up getting the "H1" mark was the child instead of
the parent.

Merge pull request #3482 from hamishimac/next

Do not assume STDIN_FILENO is available for input from child

Merge pull request #3504 from Foxboron/morten/i3-man-path

Change config order in manpage

Merge pull request #3485 from xzfc/3412-outer-border

Draw outer header borders for all layouts

Merge pull request #3505 from stapelberg/release-docs

release.sh: save docs first

release.sh: save docs first

Otherwise, as @orestisf1993 pointed out, the saved documentation will have the
wrong version number.

Draw outer header borders for all layouts

Change config order in manpage

This brings the headline for the configuration files inline with the
recent move to XDG directories.

Signed-off-by: Morten Linderud <morten@linderud.pw>

Merge pull request #3499 from orestisf1993/docs

userguide: break long comment

travis: move (failing) ubuntu build from xenial to bionic

Ubuntu’s apt started refusing to load package files from unauthenticated
repositories, but the package for which we did that (xcb-xrm) is available in
newer versions of Ubuntu, so I just removed that part altogether.

Apparently this has been broken since April, and nobody noticed :-/

update release.sh for the 4.16 release

update debian/changelog

Merge branch 'release-4.16'

Set non-git version to 4.16-non-git.

release i3 4.16

userguide: break long comment

Update ewmh focused only when new focus is different (#3496)

Fixes #3495.

Merge pull request #3497 from stapelberg/clang-format

switch to clang-format-4.0

switch to clang-format-4.0

clang-format-3.8 is old enough to have vanished from Debian testing, which we
use for our CI.