Add "modifiers" to events sent by i3bar

Merge pull request #3475 from Gravemind/fix-i3bar-re-hidden-on-any-modifier

i3bar: Fix i3bar re-hidden by any modifier (#3474)

Merge pull request #3452 from orestisf1993/title_align

Add title_align config directive

Merge pull request #3451 from orestisf1993/tray

Close & reopen tray selwin when needed

Close & reopen tray selwin when needed

Fixes #1329
Fixes #3317 (duplicate)
Examples where the tray "disappeared":
- Start without a "tray_output", reload config with "tray_output none",
remove "tray_output none" and reload.
- Start with "tray_output primary" (or "tray_output <output>"), disable
the primary output and then reconnect it.

Fixes #2010
Cached in output_for_tray

Merge pull request #3469 from yablonsky/patch-1

Update userguide "Focus Parent": add the default

Merge pull request #3448 from orestisf1993/sticky

Fix sticky focus when switching to workspace on different output

Merge pull request #3445 from orestisf1993/flaky

Fix flakyness in t/189-floating-constraints.t

Merge pull request #3444 from orestisf1993/move

Some move.c fixes

Merge pull request #3433 from orestisf1993/janitorial

Janitorial

i3bar: Fix i3bar re-hidden by any modifier (#3474)

The bug was introduced in 4.15-next fe6b3b7474

add missing parenthesis

Update userguide "Focus Parent": add the default

While reading the UserGuide, I had to refer to the TL;DR image on top to find the default keybind for "Focus Parent" since it's not mentioned in the text.
This pr fixes it.

Merge pull request #3467 from orestisf1993/userguide

userguide: Use anchor for list of commands

userguide: Use anchor for list of commands

Fixes #3464

debian/control: remove dependency on x11-utils (#3455)

This was a introduce in commit a61e34d2771fffdf2d94ee69d887585087c76b98 in 2009,
and we haven’t used xmessage in any way in a long time.

fixes https://bugs.debian.org/910997

Merge pull request #3456 from stapelberg/travis

travis: fix mk-build-deps call by providing changelog file

travis: fix mk-build-deps call by providing changelog file

mk-build-deps started using the changelog file to get the version number, but a
bug prevents it from falling back correctly to 1.0 if no changelog is
present. This has been fixed upstream in
https://salsa.debian.org/debian/devscripts/commit/4b15abd4f0cfe4accbf28f24546891867344a73c,
but we can just ship the changelog file until that fix lands.

Fix typo: childs -> children

floating_reposition: avoid extra tree_render

Use open_floating_window arguments for rect

manage_window: cleaner variable declarations

borders_to_hide doesn't need to be initialized

root is externally declared

Use lround instead of (long)round

Make comment style more consistent

Fix missing prototypes

i3 will now compile with no warnings when -Wmissing-prototypes is used.

Enforce strict prototypes

i3 will now compile with no warnings when -Wstrict-prototypes is used.

Use path_exists

Reduce some code around frees

Merge pull request #3454 from alanbarr/cppcheck_fixes_2

Cppcheck Fixes

Fixes for undefined behaviour on signed shift (#3453)

Fixes for undefined behaviour on signed shift

Change literal 1 to unsigned to allow safe bitshift of 31.
Caught by cppcheck.

Make 0xFF unsigned to prevent a left shift into signed bit.
Spotted by @orestisf1993

Typecast void* before doing pointer arithmetic

Caught by cppcheck

Remove redundant NULL check

copy has been used before this point - so it is too late to be concerned
about a NULL pointer now.

This is OK as sstrdup() calls err() on NULL return from the underlying
strdup() call.

Raised by cppcheck.

Add title_align config directive

Controls the window titles alignment in title bars. Possible values are:
- left
- center
- right

Co-authored-by: Orestis Floros <orestisf1993@gmail.com>
- Made title_align a config directive instead of a command. Helps with
some tree_render() issues we had.
- Made title_max_width the same for all 3 cases.
- Modified title offset calculations and added explanations for each
case.
- Append title_padding to mark_width if a mark exists.

Fixes #1750

Fix sticky focus when switching to workspace on different output

See the testcase for the exact steps to reproduce the problem.

output_push_sticky_windows: Make a bit easier to understand

285-sticky.t: Use kill_all_windows

Fix flakyness in t/189-floating-constraints.t

Related to #3009.

tree_move: Don't change focus order when swapping containers

The call to ewmh_update_wm_desktop is removed since the change happens
under the same parent.

move.c: Fix move_to_output_directed problems

- Use workspace_show that correctly updates _NET_CURRENT_DESKTOP, warps
mouse.
- Use TAILQ_INSERT_TAIL only for focus_head. Focus order is not related
to direction.
- Call con_focus only if con was focused before. See testcase for
directional move with command criteria.
- Correct first call of move_to_output_directed in tree_move which
didn't call ipc_send_window_event("move", con) and
ewmh_update_wm_desktop().
- Don't produce events when the move doesn't happen. Correct
276-ipc-window-move.t as well.

i3-nagbar: add option for button that runs commands without a terminal (#3258)

Fixes #2199.

Merge pull request #3441 from stapelberg/template

add specific GitHub issue templates

Merge pull request #3440 from stapelberg/disable

configure.ac: add conditionals for building docs/mans

add specific GitHub issue templates

I learnt about this from the GitHub blog:
https://blog.github.com/2018-05-02-issue-template-improvements/

resolve_tilde: strncpy + strlen is pointless (#3436)

strlen already assumes that the string is NULL-terminated.

Like in https://github.com/i3/i3status/pull/312 but for whatever reason
gcc didn't warn about this here.

userguide: Mention know issues for assign (#3434)

Fixes #3222
Fixes #3293
Related to #2060

configure.ac: add conditionals for building docs/mans

fixes #3378

Merge pull request #3435 from vivien/i3-msg/subscribe

i3-msg/subscribe

i3-msg: add support for SUBSCRIBE message type

If i3-msg is invoked with -t subscribe, it will wait for the first event
matching the given payload, before exiting.

For instance, get the number of the next focused workspace with:

    i3-msg -t subscribe '[ "workspace" ]' | jshon -e current -e num

Like inotifywait, the -m flag allows to wait indefinitely for events,
instead of exiting right after receiving the first one.

For example, continuously monitor the names of focused windows with:

    i3-msg -t subscribe -m '[ "window" ]' | jq .container.name

i3-msg: check reply in quiet mode

i3-msg currently exits right after sending the IPC message if the quiet
flag is set. This means that if an error occurred when issuing a
command, e.g. "i3-msg -q foobar", it gets silently ignored.

What we really want is to just skip printing but still check the reply.

At the same time, explicitly print the reply when we need to, instead of
using an exit label.

Merge pull request #3430 from Synray/aspect_ratio

[RFC] Correctly calculate max_aspect

Correctly calculate max_aspect

Merge pull request #3415 from orestisf1993/g_utf8_make_valid

Check g_utf8_make_valid availability

Merge pull request #3424 from orestisf1993/con_num_windows

con_num_windows: Count floating windows

con_num_windows: Count floating windows

Fixes #3423.

Merge pull request #3421 from Streetwalrus/iconic-state

Reject requests for WM_STATE_ICONIC

Reject requests for WM_STATE_ICONIC

For compatiblity reasons, Wine will request iconic state and cannot
ensure that the WM has agreed on it; immediately revert to normal to
avoid being stuck in a paused state.

Provide g_utf8_make_valid if not available

See #3415 for licensing discussion.

Fixes Airblader/i3#236

Fix typo: terminaison -> termination

Merge pull request #3420 from orestisf1993/sncontext==NULL-3419

i3-nagbar & i3-config-wizard: check sncontext != NULL

i3-nagbar & i3-config-wizard: check sncontext != NULL

From
https://github.com/freedesktop/startup-notification/blob/07237ff25d6171e1b548118442ddba4259a53ba5/libsn/sn-common.c#L87-L171
it appears that SnDisplay can't be NULL, so I skipped the check.

Fixes #3419

Merge pull request #3416 from orestisf1993/530-bug-2229.t

530-bug-229.t: Get rid of smartmatch

530-bug-229.t: Get rid of smartmatch

Merge pull request #3400 from Synray/next

Respect max size from WM_NORMAL_HINTS

Respect max size in WM_NORMAL_HINTS

Merge pull request #3408 from orestisf1993/__focused__-crash

match_matches_window: Check if focused->window exists

Merge pull request #3410 from orestisf1993/min_width

docs/i3bar-protocol: Mention skipping blocks with empty full_text

Merge pull request #3386 from orestisf1993/click-event

Fix "relative_x" and "width" of click events

docs/i3bar-protocol: Mention skipping blocks with empty full_text

Closes #3405.

Fix "relative_x" and "width" of click events

Now clicks begin at the start of the "actual" block, offsets and
separators don't trigger click events. The width property is now just
the width of the block, including border.

Fixes #3380.

match_matches_window: Check if focused->window exists

Fixes #3406.

Merge pull request #3409 from orestisf1993/sb_hoff_px

i3bar: Include sb_hoff_px only when needed

i3bar: Include sb_hoff_px only when needed

Fixes #3404.

Merge pull request #3403 from orestisf1993/issue-3402

_con_move_to_con: Move upwards only on CT_FLOATING_CON

_con_move_to_con: Move upwards only on CT_FLOATING_CON

If target is inside a floating container but not its direct child, the
move can be completed as is.

Fixes #3402.

Merge pull request #3401 from orestisf1993/dump-asy-floating

dump-asy.pl: Include floating_nodes

dump-asy.pl: Include floating_nodes

Merge pull request #3342 from orestisf1993/tree_close_internal

Simplify tree_close_internal

Merge pull request #3399 from orestisf1993/156-fullscreen-focus

Make t/156-fullscreen-focus.t easier to work with

Merge pull request #3398 from orestisf1993/i3-save-tree

i3-save-tree: Exclude unsupported "transient_for" property

Make t/156-fullscreen-focus.t easier to work with

- Add routine that will refocus the expected window on test failure.
Thus, failure on one test will not make others fail.
- Remove some redundant commands, prefer fresh_workspace for screen
changing.
- Kill previous windows between sections if the next section does not
depend on the previous layout.

i3-save-tree: Exclude unsupported "transient_for" property

Even if i3 supported matching windows through "transient_for", it
wouldn't be useful for the purpose of i3-save-tree.

Merge pull request #3395 from lousyd/next

clarify JSON standard non-compliance

Merge pull request #3396 from orestisf1993/commands

commands.c: Improve error replies

commands.c: Improve error replies

- Improve / add various error messages.
- Replace all `LOG(…); ysuccess(false);` with `yerror(…);`.
- switch_mode: Remove redundant "ERROR:" ELOG string.
- cmd_move_con_to_workspace*: Make sure that we don't try to move an
empty workspace to another workspace. This can be problematic when we
match a workspace using command criteria (eg marks) and the target is a
non-existing workspace. We create the new workspace but since nothing is
moved there, we are left with an empty workspace. See added testcase.

clarify JSON standard non-compliance

Modified section on the layout file's non-compliance with the JSON
standard. The section previously stated that having multiple top-level
JSON texts is non-compliant. This isn't the case. It's just that most
JSON parsers will treat that as if it is non-compliant.

Merge pull request #3394 from orestisf1993/validate-utf8

 libi3: validate UTF8 strings

libi3: validate UTF8 strings

Will validate container / window titles.

Fixes #3156.

tree_restore: Check croot

Related to #2414, #3156.

Merge pull request #3391 from orestisf1993/yajl-utf8

tree_append_json: Allow strings that are not valid UTF8

tree_append_json: Allow strings that are not valid UTF8

Fixes #3156.

I couldn't reproduce the problem in a "natural" way so I cheated:
1. Start i3 with gdb
2. Set breakpoing on tree_restore
3. Run, open window, i3-msg restart
5. Open the file in *path with a hex editor
6. Edit the "name" field of the window and insert bytes that are not
valid UTF8
7. Continue

After parsing fails, all nodes including croot are incomplete, meaning
they have to be deleted. We can't recover in any reasonable way so we
have to allow non-UTF8 characters to avoid this situation altogether.

tree_append_json: don't focus freed container

Merge pull request #3389 from orestisf1993/resize-ensure-1px

Ensure containers have a size of at least 1px after resize

Merge pull request #3390 from orestisf1993/resize-tile

resize set for tiling: default to px when not specified

Merge pull request #3388 from orestisf1993/cmd_rename_workspace

cmd_rename_workspace: correct order of events

resize set for tiling: default to px when not specified

Ensure containers have a size of at least 1px after resize

Fixes #2226.
Fixes #2776.
Fixes #3241.
Related to #3194.

cmd_rename_workspace: correct order of events

1. Rename happens
2. Workspace is moved because of assignments
3. Workspace closes because it is empty (#3248)

Fixes #3248.

Improve comment from #3245