Revert "Workspace command number selection"

This reverts commit cbd51ce6642be4e1163a64dce8ec20a454f1bf05.

If you’ve started using this feature, I’m very sorry, but I think it’s
better to revert it entirely, as it will only cause more problems in the
future as more and more people upgrade and get bitten by this if we
didn’t revert it…

fixes #1536

Merge pull request #1564 from acrisci/feature/i3bar-markup-per-block

i3bar: set markup per block

i3bar: set markup per block

Add `markup` to the i3bar protocol as a block member.

This is a string that determines how the block should be parsed as
markup. "pango" indicates the block should be parsed as Pango markup.
"none" indicates the block should not be parsed as markup.

Merge pull request #1570 from shdown/next

i3bar: spelling fixes

i3bar: spelling fixes

Merge pull request #1562 from Airblader/feature-short-text

Add support for the short_text property

Merge pull request #1568 from acrisci/feature/valgrind-no-verbose

testcases: dont run valgrind verbose

testcases: dont run valgrind verbose

Add support for short_text

When the statusline is too long, try to use the short_text property of each status block before falling back to truncating it.
fixes #1092

Introduce a function to copy i3strings

Introduce the short_text property on blocks and parse it according to the documentation.

clang-format and rearrange comments in i3bar/src/xcb.c

fix travis config with regards to clang-format

Merge pull request #1560 from shdown/refactoring

i3bar: refactoring

Merge pull request #1557 from acrisci/bug/fix-validation-segfault

Bugfix: validation segfault

i3bar: introduce get_tray_width() function to reduce duplication

i3bar: refactor magic numbers to *_px constants

Bugfix: validation segfault

When `new_window` is given in the config, config validation with `i3 -C`
would segfault.

Add a NULL check in logical_px() to check for the case when the config
is being validated without an X server to prevent this.

Don’t overwrite border width when already set (placeholders).

fixes #1501

i3bar: set correct initial position when reconfiguring

fixes #1542

Merge pull request #1556 from Airblader/feature-i3-save-tree-by-number

Allow referencing the workspace by number.

increase timeouts for travis

See https://travis-ci.org/i3/i3/jobs/55319961 (line 4064) for a job
where the window could not be mapped within 2 seconds.

Merge pull request #1561 from Airblader/feature-append-layout-docs

Added a troubleshooting section to the restoring layout docs

Added a troubleshooting section to the restoring layout docs and described solution for vertically split containers

Merge pull request #1559 from shdown/next

various i3bar bugfixes

i3bar: fix click events for workspace buttons with long statusline

This combines detecting of current and clicked workspaces into one cycle
and then checks if the x coordinate of the clicked point is greater than
the workspace buttons width.

i3bar: fix spelling is messages

i3bar: rename a variable

Give it a more meaningful name.

i3bar: add binding indicator width to workspace buttons width

Just to prevent the indicator from being overlapped by the statusline.

Allow referencing the workspace by number.

This allows "--workspace 3" for numbered workspaces, e.g., if the name is actually
"3:foo". This introduces the same functionality the IPC already offers in many places.

i3bar: cut long statuslines from the left

This draws the statusline after drawing (and calculating width of) the
workspace buttons and fixes calculation of its maximum visible width.
This (hopefully) guarantees that these two will never overlap.

i3bar: fix clearing of the statusline pixmap

This fixes the bug when statusline pixmap wasn't entirely cleared that
caused random artifacts to appear when the statusline width is greater
than the screen width.

Merge pull request #1555 from Airblader/feature-i3-save-tree-current-ws

Make the --workspace optional by defaulting to the focused workspace.

Merge pull request #1549 from shdown/y-offset-fix

Fix incorrect y-offset for text in i3bar (2)

adapted usage text

Make the --workspace optional by defaulting to the focused workspace.

If no option is given, i3-save-tree should default to the currently focused workspace.
Specifying both --workspace and --output will still yield an error.

Merge pull request #1551 from acrisci/bug/rename-workspace-mismatch

bugfix: workspace rename focus mismatch

Merge pull request #1553 from Airblader/feature-remove-output-assignment

Remove the A_TO_OUTPUT option

fix warnings when CONTINUOUS_INTEGRATION env var is unset

relax timings of t/200-urgency-timer.t to reduce flakiness

Remove the A_TO_OUTPUT option as it is not needed anymore and will not be implemented in favor of normal assignments.

bugfix: workspace rename focus mismatch

When renaming a workspace on another output that moves the workspace to
the current output, the renamed workspace would be shown even though it
does not contain the focused container.

Explicitly show the focused workspace after the move. This is necessary
because `workspace_move_to_output` will show the workspace that is
moved.

Fix incorrect y-offset for text in i3bar (2)

014aa7ff7441a8a2bb53bd0956aff61faaf6f37f fixed incorrect align of small
symbols, but also introduced a problem with texts with mixed large and
normal symbols. Fix it by centering the text vertically only if its
height is smaller that the saved font height.
Fixes #1543.

travis: verify code was formatted using clang-format

clang-format src/commands.c

Merge pull request #1539 from mh21/wsrename-1527

Rename workspaces in startup sequences

travis: cat complete-run.log on failures

tests: exit with status 1 when tests fail

tests: don’t print interactive status when running on travis

(or without a tty)

travis: run tests serialized, install latest AnyEvent::I3

travis: install as many perl pkgs as possible via apt instead of cpanm

travis: fix pinning setup (newlines and YAML are complicated)

travis: escape strings for YAML

travis: install more perl dependencies, pull in trusty packages

travis: also cat cpanm build logs after failure

travis: cpanm: set -v so that we get error messages in the travis output

Rename workspaces in startup sequences.

When renaming workspaces, any workspace names in pending startup
sequences also need to be renamed.

Test: startup workspaces updating on rename.

travis: directly use cpanm -n to skip tests

X11::XCB currently fails its tests

travis runs on ubuntu 12.04, so use libinline-perl

travis: mk-build-deps needs equivs

add configuration for travis-ci.com

Merge pull request #1521 from Airblader/feature-use-refactored-function

Use the already existing function to get rid of some code duplication

Merge pull request #1513 from Airblader/defect-1473

Move workspace to assigned output when renaming it

Merge pull request #1512 from mdtipton/unclutter_scroll

Fix scrolling in window decoration with hidden cursor.

Merge pull request #1546 from rosetree/next

Fix comment box

Fix comment box

Merge pull request #1545 from garbados/gendered-terms-in-docs

degendered terms

degendered terms

add release.sh, a script to prepare a new i3 release

update debian/changelog for the new release

Merge branch 'master' into next

Merge branch 'release-4.9.1'

release i3 4.9.1

Merge branch 'master' into next

Conflicts:
src/commands.c

Merge branch 'fix-restore-errormsg'

Properly error out when the layout file cannot be read.
This will result in an actual error message for the user.
fixes #1499

Glob filepath when calling append_layout
fixes #1500

Improve error messages on failing commands

Merge branch 'master' into next

Merge branch 'dont-match-placeholders'

Bugfix: insert id-based match at HEAD, not TAIL.

Before this commit, placeholder windows had any matches that were
defined in the JSON file, _followed_ by an i3-internal match that
ensures the placeholder X11 window gets swallowed into the placeholder
i3 container.

The problem was that the first successful match was deleted, and if
users specified a criterion (title=IPython) that matched the placeholder
window itself (name=IPython), then that match is deleted and the
i3-internal match is kept. This results in the actual window the user
wants to match not being swallowed, and the placeholder window not
displaying any criteria.

fixes #1526

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

Use the already existing function to get rid of some code duplication

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

Improve error messages on failing commands

Improve error messages on failing commands

Added tests for #1473

Refactor functions for easy reuse

When renaming a workspace, look for assignments and move the renamed workspace to the appropriate output.

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

Fix scrolling in window decoration with hidden cursor.

If the mouse cursor is hidden (by unclutter, for example), then scrolling
in the window decoration creates an event with a child
(i.e. event->child != XCB_NONE). This causes route_click() to be called
with dest=CLICK_INSIDE, which prevents scrolling through a stacked layout.

To fix this, check if a click is in the window decoration _before_
checking if the event has a child.

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

Glob filepath when calling append_layout

Merge branch 'Airblader-defect-1480' into next