Merge pull request #2820 from hwangcc23/fix-2804

i3-msg: Return an exit code upon missing -t arg

i3-msg: Return an exit code upon missing -t arg

If getopt() returns '?' (option with a missing argument), exit the program with an error code rather than continuing the execution.

Fix the issue #2804 (https://github.com/i3/i3/issues/2804).

Merge pull request #2808 from hwangcc23/fix-2802

Fix 'rename workspace to tosomething'

Fix 'rename workspace to tosomething'

This patch fixes the issue #2802 (https://github.com/i3/i3/issues/2802).

1). Revise the state machine for the 'rename workspace' command.
    These scenarios are considered:
    a). 'rename workspace to to bla'
        state transitions: RENAME -> RENAME_WORKSPACE -> RENAME_WORKSPACE_LIKELY_TO -> RENAME_WORKSPACE_LIKELY_TO_NEW_NAME
    b). 'rename workspace to tosomething'
        state transitions: RENAME -> RENAME_WORKSPACE -> RENAME_WORKSPACE_LIKELY_TO
    c). 'rename workspace to to'
        state transitions: RENAME -> RENAME_WORKSPACE -> RENAME_WORKSPACE_LIKELY_TO
    d). 'rename workspace to bla'
        state transitions: RENAME -> RENAME_WORKSPACE -> RENAME_WORKSPACE_LIKELY_TO
    e). 'rename workspace bla to foo'
        state transitions: RENAME -> RENAME_WORKSPACE -> RENAME_WORKSPACE_TO -> RENAME_WORKSPACE_TO_NEW_NAME

2). Add a test case in 117-workspace.t for the scenario b.

Merge pull request #2795 from Adaephon-GH/doc-config-load-order

Fix manpage on configuration lookup order

Merge pull request #2799 from hwangcc23/add_compilation_inst

Add compilation instructions to the hacking howto

Add compilation instructions to the hacking howto

Merge pull request #2796 from Armael/next

Add a link to the ocaml-i3ipc library

Add a link to the ocaml-i3ipc library

Fix manpage on configuration lookup order

The lookup order stated in the i3 manpage was is contratiction to the
actual lookup order, which was introduced with commit
https://github.com/i3/i3/commit/bfa12a581915d6a3de182fa6025fce108cac8eab.
Since that commit (6.5 years ago) the "traditional" paths took precedence
over XDG_CONFIG paths.

Merge pull request #2772 from loungecube/patch-1

prevent multiple menu items per .desktop entry

Merge pull request #2784 from stapelberg/numrelease

compare modifiers for equality, not subset

compare modifiers for equality, not subset (+test)

Subset comparison was introduced with the rather large commit
bf3cd41b5ddf1e757515ab5fbf811be56e5f69cc, but I now think we should use
equality.

In other words, the following key binding:

    bindsym Mod4+x nop Mod4+x

previously would have been triggered when pressing Mod3+Mod4+x.

Strictly speaking, this is a change of behavior, but it breaks none of our
tests, and using equality instead of subset comparison enables more use-cases.

fixes #2002

no-op change: store |button| in the correct data type

no-op change: move bind->release check into loop

We don’t have to negate the check anymore, making it more readable.

no-op change: don’t compare keycode for every modifier

Fix the i3 crash caused by mark + restart commands (#2779)

This patch fixes the issue #2511(https://github.com/i3/i3/issues/2511).

1). Memorize the marks, but only call con_mark once the container has finished parsing. (Credit: This is @Airblader's patch.)

2). Add a test case 267-regress-mark-restart.t for regression test to check if mark and restart command crash i3.

Merge pull request #2782 from stapelberg/release-bindings

Fix --release bindings broken by previous commit

Fix --release bindings broken by previous commit

fixes #2559

Apply numlock fallback to bindcode where necessary (#2781)

This was broken with commit d77d40173aae250d2e5ba86a8bce7a6e857cbc67

fixes #2559

Merge pull request #2777 from hwangcc23/update-testsuite-doc

Update testsuite docs

Update testsuite docs

After moved to autotools, the method to run the i3 testsuite is changed.
Update the testsuite document to reflect the change.

Merge pull request #2773 from lasers/next

docs/ipc: Fix typo

docs/ipc: Fix typo

Merge pull request #2761 from mdirkse/next

[WIP] Change golang ipc lib reference to one that is maintained.

Merge pull request #2771 from hwangcc23/fix-2764

Support to get the primary output

Support to get the primary output

This makes `primary` output available for assign or move commands.
Fix the issue #2764(https://github.com/i3/i3/issues/2764).

Merge pull request #2496 from Airblader/feature-917

Implement 'swap' command.

Implement 'swap' command.

This patch introduces a new command 'swap' that swaps two containers so
that they assume each other's position and geometry.

fixes #917

prevent multiple menu items per .desktop entry

If a .desktop entry is being added to the list of Menu items by its "Name" field, it should not be added again by its command or filename.
If it is being added by its command, it should not be added again by its filename.

Force container to be redrawn upon moving to another container. (#2769)

This is necessary as otherwise urgent containers can be broken
after moving them.

Merge pull request #2766 from MachFour/nagbar-shell-exec

i3-nagbar: add button flag to execute action with /bin/sh directly

i3-nagbar: add button flag to execute action with /bin/sh directly.
Fixes #2765.

add error check and log for xcb_create_window

Added a hint about the required config v4 version hint. (#2759)

fixes #2751

Merge pull request #2762 from mdirkse/valid_json

Fix invalid JSON

Fix invalid JSON

Change golang ipc lib reference to one that is maintained.

Merge pull request #2757 from Jereq/next

Fix i3-dmenu-desktop quoted command name

Query workspaces again in i3bar when an output change occured. (#2760)

As of 2f0f8b1, i3bar will properly clean up on output change events.
However, this requires us to query the workspaces again to avoid a
display error in i3bar.

fixes #2740
fixes #2743

Introduce --exclude-titlebar flag for mouse bindings. (#2703)

This introduces the flag --exclude-titlebar for mouse bindings which
allows bindings like

    bindsym --whole-window --border --exclude-titlebar button3 focus

fixes #2347

Fix i3-dmenu-desktop quoted command name

According to the Desktop Entry Specification
https://specifications.freedesktop.org/desktop-entry-spec/desktop-entry-spec-latest.html#exec-variables
the executable name or path of the executable may be quoted. This is not
properly respected when i3-dmenu-desktop extracts the command name from
the Exec entry.

Examples of values that fail and what they currently result in:

- "bar" -> "bar"
- "foo/bar" -> bar"
- "foo foobar/bar" -> "foo
- "foo\sbar" -> "foo\sbar"
- foo\sbar -> foo\sbar
- "foo\\\\bar" -> "foo\\\\bar"

Merge pull request #2752 from mihaicmn/bug-2464

Fix changing of root workspace layout from stacked/tabbed

Fix changing of root workspace layout from stacked/tabbed

Merge pull request #2731 from nmschulte/check-create-window

Check X11 create window call

add error check and log for xcb_create_window

Merge pull request #2725 from nmschulte/feature/output-primary

Feature/output primary

format i3bar src/outputs.c w/ clang-format

Merge pull request #2720 from Arlon1/next

Adding new terminals to i3-sensible-terminal

Adding new terminals to i3-sensible-terminal

i3bar: accept 'primary' for output config option

Merge pull request #2698 from tmerr/next

docs/ipc: Document the 'primary' flag

docs/ipc: Document the 'primary' flag

Resolves #2697

Fixes a small mixup in the assertion description. (#2692)

relates to PR #2649

Merge pull request #2649 from s3rb31/next

layout toggle: take any combination of layouts as arguments (continuation of #2476)

layout toggle: take any combination of layouts as arguments (#2649)

With this PR the 'layout toggle' command can be passed any
combination of valid layout keywords as arguments. They will
be activated one after another each time you issue the command,
advancing from left to right always selecting the layout after
the currently active layout or the leftmost layout if the active
layout is not in the argument list.

This PR also incorporates the feature request from #2476.

Implement mapping from string to layout as extra function

Merge pull request #2680 from Airblader/feature-2677

[#2677] Allow using left/right scrolling like up/down scrolling.

Copy the entire window content on Expose events. (#2685)

With commit d58dbc3 we started ignoring Expose events in a sequence
except for the last one. Since we only copied the affected part of
the window in the Expose event handler, this caused incorrectly
rendered window decorations.

Instead of reverting to the old behavior, we now copy the entire window
content on this single, last event with the following rationale:
- It's cheaper to copy a larger chunk once than multiple smaller
  chunks doing one server roundtrip each.
- That's how we do it when rendering out decoration on decoration
  changes as well.

fixes #2683

Allow using left/right scrolling like up/down scrolling.

This commit makes left/right scrolling synonyms for up/down scrolling for
* scrolling on window decoration
* scrolling on i3bar workspaces

fixes #2677

Introduce named aliases for mouse buttons.

This increases readability and allows us to cover up the fact that
XCB doesn't define constants for left/right scrolling.

Add troubleshooting for title attributes during layout restoring. (#2679)

document our project governance model (#2675)

Explaing the workspace number "1: www" behavior (#2674)

The documentation did not explain how workspace number `"1: www"` is working.
Related to #2663

Rephrase to cover the creation case

docs: focus_follow_mouse only happens at window border crossings (#2669)

see #2666

restart bar status command on config reload (#2668)

Closes #2651

migrate i3-config-wizard to draw_util (#2654)

Added missing cases for workspace event (#2656)

The possible values "rename", "reload" and "restored" of the property
'change' from the workspace event were missing. Because no events of
those types contain an old workspace, this was trivial.

Testcases: Check for required binaries (#2655)

Fail fast in case test binaries are not built and provide instructions
on how to build them.

Add the ipc shutdown event (#2652)

This event is triggered when the connection to the ipc is about to
shutdown because of a user action such as with a `restart` or `exit`
command. The `change` field indicates why the ipc is shutting down. It
can be either "restart" or "exit".

fixes #2318

migrate placeholder windows to draw_util (#2646)

migrate i3-input to draw_util (#2645)

migrate i3-nagbar to draw_util (#2644)

Update userguide (#2647)

key identifier Esc is invalid, use Escape instead

Ensure that marks and the title are displayed even if the window title is empty. (#2639)

Previously rendering marks and the title were skipped if the title is empty. With marks
this is obviously wrong, with the title it is also wrong because title_format might be
set.

Don't exit() on redefined binding mode. (#2638)

Doing a hard exit() is a rather harsh action for something i3 can handle
perfectly fine and is only meant to be a check to make debugging easier
for users in certain situations.

Only react on the last Expose event in a series of events. (#2636)

Thanks to @psychon for pointing this out during the review of PR #2624.
This commit extends this change to all other occurences of Expose events
within i3.

Fix blinking test. (#2637)

Thanks to @sandsmark for figuring this out.

Remove unused function arguments. (#2635)

The connection is no longer necessary since the non-cairo paths have
been removed.

Validate that a binding mode is not defined more than once. (#2633)

While defining the same mode usually wouldn't hurt and, in fact, the old behavior
allows to split the definition of a binding mode into several blocks, this
can lead to user errors where they accidentally define a mode twice and don't understand
why the mode behaves a certain way (this has been observed in real life :-)).

There's no good usecase for splitting a single binding mode into multiple blocks, thus
the new behavior is better.

fixes #2615

Added support for _NET_MOVERESIZE_WINDOW. (#2634)

fixes #2603

Free trayclient when removing it. (#2632)

fixes #2619

Merge pull request #2624 from Airblader/bug-2422

Rewrite the signal handler dialogs.

Merge pull request #2629 from nmschulte/next

fix conditional in configure script

fix conditional in configure script

Rewrite the signal handler dialogs.

This commit is a rewrite of the popup dialogs used when i3 crashes. We now
use our draw_util suite and both properly react to EXPOSE events and clean
up the windows when the handler exits.

As a side-effect, this fixes #2422

Merge pull request #2628 from nmschulte/fix_build_outside_repo

Fix build outside repo

Report error during error log creation (#2625)

linking vim_like_marks from show_marks documentation (first occurrence (#2626)

of marks in userguide)

fix auto exclude in-work-tree build dirs

updates: -- configure: add build directory to gitignore #2543 -- https://github.com/i3/i3/pull/2543

properly detect version when building out of tree

.github/CONTRIBUTING.md: explain that compositors are unsupported (#2621)

Free allocated X resource value. (#2620)

This fixes a little memory leak.

Fix name of mcedit (#2524)

It's "mcedit", not "mc-edit".

Some systems need to link to libiconv explicitly, thus add a check for it. (#2586)

Do not set input focus in i3-input. (#2598)

This commit removes all traces of setting and reverting the input focus
in i3-input. We don't need to do this because grabbing the keyboard is
sufficient to have the attention we need.

Changing the input focus and reverting it can cause situations where i3
executes the IPC command before processing the FocusIn events. This leads
to i3's input focus change to be rejected due to the timing, leading to
an inconsistent focus state.

fixes #2597

Fix read of uninitialized memory (#2596)

Previous code was reading whole array, it was slower and it
read uninitialized memory

`move` syntax clarification: (#2591)

both x and y position need to be specified

Respect minimum size hints for floating windows. (#2508)

This commit introduces proper support for the minimum size on floating
windows by ensuring that it is respected during mapping, later changes as
well as resizes.

Furthermore, this commit fixes minor issues with how the hints are handled
during calculations.

fixes #2436

Use the DPI setting within the i3-config-wizard and i3-nagbar (#2585)