kill left-over variable

Use only 'import' of 'Exporter'

reformat perldoc paragraph

use constant instead of magic number

Get rid of _bytelength, use encode_utf8 and length instead. Correctly check for scalar

Bugfix: Handle message_type == 0

Add new constants

update MANIFEST correctly (use MANIFEST.SKIP instead of ignore.txt)

Update MANIFEST

Provide constants for message types

Expand testcase for the get_workspaces message (needs i3 to be running)

Return connection status in condvar in $i3->connect

kill boilerplate

prefix internal subs with _

Initial commit

t/171-config-migrate: escape braces for newer Perl versions

Prevent accidentally introduced fall-through

This was added in commit https://github.com/i3/i3/commit/e82e26a24d22e2f92cd90d9547c213493e94ea63

Merge pull request #2847 from tmerr/next

Document missing case of workspace event

Document missing case of workspace event

The docs are missing the "move" workspace event emitted from
workspace.c's workspace_move_to_output function.

Add missing newline to end of file

Merge pull request #2836 from acrisci/bug/sysconfdir-to-makefile

Move SYSCONFDIR definition to makefile

Merge pull request #2838 from acrisci/bug/colors-only-hex

Ensure colors are in hex

libi3: Add basic validation to hex color conversion

Make sure a given hex color is the expected length and begins with a hash in
draw_util_hex_to_color() to avoid memory errors.

fixes #2829

i3bar: change error block color to hex

Named colors are not supported by the i3bar protocol so give the error block
color in hex.

Move SYSCONFDIR definition to makefile

Official autoconf docs contraindicate the use of the $sysconfdir
variable in configure.ac. For reference:

https://www.gnu.org/software/autoconf/manual/autoconf-2.63/html_node/Installation-Directory-Variables.html

The macro SYSCONFDIR (ordinarilly set to "/etc") is required to find
a system configuration file installed with i3. In some build setups,
this may not be defined properly in when the configure script is
compiled.

Instead, define this variable in AM_CPPFLAGS as the documentation
indicates.

fixes #2832

Merge pull request #2831 from dvejmz/next

Add KDE konsole terminal fallback to i3-sensible-terminal

Add KDE konsole terminal fallback to i3-sensible-terminal

Merge pull request #2827 from akash-akya/next

Fix 'Mouse binding don't work'

Fix 'Mouse binding don't work'

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)