| 4e663ab | Drew DeVault | 14 June 2017, 23:02:15 UTC | Merge pull request #1199 from SirCmpwn/server-decoration Implement KDE's server-side decoration protocol | 14 June 2017, 23:02:15 UTC |
| 298f563 | Drew DeVault | 14 June 2017, 22:53:40 UTC | Merge branch 'master' into server-decoration | 14 June 2017, 22:53:40 UTC |
| eb6e38c | Drew DeVault | 14 June 2017, 22:53:32 UTC | Merge pull request #1234 from 4e554c4c/tray Implement Tray Icons | 14 June 2017, 22:53:32 UTC |
| 0fc9628 | Drew DeVault | 14 June 2017, 22:14:22 UTC | Merge branch 'master' into tray | 14 June 2017, 22:14:22 UTC |
| 76fcfa1 | Drew DeVault | 14 June 2017, 22:13:41 UTC | Force focus on unmanaged views during pointer test | 14 June 2017, 22:13:41 UTC |
| d158e18 | Drew DeVault | 14 June 2017, 22:09:05 UTC | Fix Firefox address bar thing And probably some other stuff too | 14 June 2017, 22:09:18 UTC |
| 33fdae2 | Calvin Lee | 13 June 2017, 19:42:11 UTC | Remove Xembed Support Xembed support is premature in sway and should be postponed. This commit only removes swaybar starting xembedsniproxy, if users would like, they can still start xembedsniproxy manually, however there will be no official support. | 13 June 2017, 19:42:11 UTC |
| 16e20ec | Drew DeVault | 12 June 2017, 22:03:25 UTC | Merge pull request #1236 from ykrivopalov/fix_complex_status_command swaybar: Group child processes for signalling | 12 June 2017, 22:03:25 UTC |
| 8f4bc0a | Yury Krivopalov | 12 June 2017, 20:01:19 UTC | swaybar: Group child processes for signalling Fixes child proccess killing, when status_command is a complex command like "i3status | wrapper.sh". Set the process group id of the child process by calling `setpgid` after forking and before calling `exec`. The process group ID will be set to the process ID of the forked process. Processes spawned by this child process will also have this group ID. Send signals to the process group with `killpg`. This will send the signal to all of the process group. | 12 June 2017, 20:12:31 UTC |
| 0a71aa6 | Calvin Lee | 08 June 2017, 12:36:17 UTC | Fix Catching NewIcon Signal The unique name was not copied out of the wire marshalled DBus message data so `sni_uniq_cmp` would always match against junk data. | 08 June 2017, 15:24:35 UTC |
| 154cc8a | Drew DeVault | 08 June 2017, 13:56:34 UTC | Merge pull request #1235 from vaartis/patch-1 include stdint to be compaitable with musl | 08 June 2017, 13:56:34 UTC |
| 4bf8b6b | Ekaterina Vaartis | 08 June 2017, 13:40:17 UTC | include stdint to be compaitable with musl | 08 June 2017, 13:40:17 UTC |
| 1451ee8 | Calvin Lee | 08 June 2017, 04:32:48 UTC | Reorganize Tray Code Remove tray code from bar.c and render.c | 08 June 2017, 04:32:48 UTC |
| 790887c | Drew DeVault | 08 June 2017, 02:45:33 UTC | Fix cairo blending on tray icons | 08 June 2017, 02:45:33 UTC |
| 843ad38 | Calvin Lee | 07 June 2017, 23:45:28 UTC | Implement Tray Icons This commit implements the StatusNotifierItem protocol, and enables swaybar to show tray icons. It also uses `xembedsniproxy` in order to communicate with xembed applications. The tray is completely optional, and can be disabled on compile time with the `enable-tray` option. Or on runtime with the bar config option `tray_output none`. Overview of changes: In swaybar very little is changed outside the tray subfolder except that all events are now polled in `event_loop.c`, this creates no functional difference. Six bar configuration options were added, these are detailed in sway-bar(5) The tray subfolder is where all protocol implementation takes place and is organised as follows: tray/sni_watcher.c: This file contains the StatusNotifierWatcher. It keeps track of items and hosts and reports when they come or go. tray/tray.c This file contains the StatusNotifierHost. It keeps track of sway's version of the items and represents the tray itself. tray/sni.c This file contains the StatusNotifierItem struct and all communication with individual items. tray/icon.c This file implements the icon theme protocol. It allows for finding icons by name, rather than by pixmap. tray/dbus.c This file allows for asynchronous DBus communication. See #986 #343 | 08 June 2017, 00:49:16 UTC |
| fd47a30 | Drew DeVault | 06 June 2017, 13:34:01 UTC | Merge pull request #1232 from johalun/master-freebsd FreeBSD fixes | 06 June 2017, 13:34:01 UTC |
| 7fef283 | johalun | 05 June 2017, 11:44:29 UTC | FreeBSD fixes Increase _POSIX_SOURCE value where needed. Increase _XOPEN_SOURCE value where needed. Conditionally link to libcap (only on Linux). Possibly some trailing whitespace fixes (automatic). | 06 June 2017, 07:45:50 UTC |
| 5ed533a | Drew DeVault | 17 May 2017, 17:11:49 UTC | Revert "Drop epoll dependency" This reverts commit 91f2bf81bf89699d43083f3d3a27805ba5e395a9. | 17 May 2017, 17:11:49 UTC |
| 91f2bf8 | Drew DeVault | 17 May 2017, 17:10:33 UTC | Drop epoll dependency | 17 May 2017, 17:10:33 UTC |
| 6df0f9a | Drew DeVault | 11 May 2017, 16:48:30 UTC | Merge pull request #1218 from Hummer12007/suicaps Terminate when both suid bit and filecaps are set | 11 May 2017, 16:48:30 UTC |
| f470fa5 | Mykyta Holubakha | 11 May 2017, 16:38:32 UTC | Replace spaces with tabs in resolve_path | 11 May 2017, 16:38:32 UTC |
| 93cf21f | Mykyta Holubakha | 09 May 2017, 23:51:28 UTC | Terminate when both suid bit and filecaps are set | 11 May 2017, 16:33:57 UTC |
| f736198 | Mykyta Holubakha | 11 May 2017, 16:29:10 UTC | Initialise logging earlier | 11 May 2017, 16:29:25 UTC |
| 46fbb41 | Drew DeVault | 11 May 2017, 15:47:42 UTC | Mention Patreon in README.md Does this make me a sellout | 11 May 2017, 15:47:42 UTC |
| ee81b1a | Drew DeVault | 06 May 2017, 12:43:48 UTC | Merge pull request #1211 from wdbw/missing-send-event Add missing send-event mode to sway-input(5) | 06 May 2017, 12:43:48 UTC |
| 264a9ee | wdbw | 06 May 2017, 09:45:17 UTC | Add missing send-event mode to sway-input(5) Also, fix a small typo. | 06 May 2017, 09:45:17 UTC |
| 781fc60 | Drew DeVault | 04 May 2017, 21:17:13 UTC | Merge pull request #1209 from janza/mouse-bindings Add mouse button bindings | 04 May 2017, 21:17:13 UTC |
| 400998d | Josip Janzic | 04 May 2017, 20:42:50 UTC | Add mouse button bindings Adds support for bindings like: bindsym button3 floating toggle bindsym $mod+button3 floating toggle | 04 May 2017, 21:00:03 UTC |
| 02d1ae7 | Drew DeVault | 01 May 2017, 16:58:02 UTC | Merge pull request #1206 from zandrmartin/document-font-command document font command | 01 May 2017, 16:58:02 UTC |
| 51ce752 | Zandr Martin | 01 May 2017, 16:53:07 UTC | document font command | 01 May 2017, 16:53:07 UTC |
| 106868d | Drew DeVault | 01 May 2017, 02:51:52 UTC | Update config.in swaymsg -t get_inputs now pretty prints its output, so this warning is obsolete | 01 May 2017, 02:52:19 UTC |
| 5e845a3 | Drew DeVault | 29 April 2017, 18:16:20 UTC | Merge pull request #1204 from zandrmartin/prevent-layout-auto-crash prevent crash when `layout auto` is missing args | 29 April 2017, 18:16:20 UTC |
| f65a88e | Zandr Martin | 29 April 2017, 18:02:07 UTC | prevent crash when `layout auto` is missing args | 29 April 2017, 18:02:07 UTC |
| 34fae76 | Drew DeVault | 29 April 2017, 14:54:08 UTC | Add HACKING.md | 29 April 2017, 14:54:08 UTC |
| 816cedd | Drew DeVault | 29 April 2017, 14:07:18 UTC | Set child border to background if not specified | 29 April 2017, 14:07:18 UTC |
| eab29ca | Drew DeVault | 29 April 2017, 14:04:13 UTC | Merge pull request #1201 from SirCmpwn/fix-colors Support specifying fewer than 5 colors | 29 April 2017, 14:04:13 UTC |
| 02812a2 | Drew DeVault | 29 April 2017, 13:56:07 UTC | Support specifying fewer than 5 colors This doesn't work, I'm not sure why. The color structs definitely get changed but if you specify fewer than 5, it renders with the defaults. | 29 April 2017, 13:56:07 UTC |
| a5c07dd | Drew DeVault | 29 April 2017, 00:32:05 UTC | Implement KDE's server-side decoration protocol | 29 April 2017, 00:32:05 UTC |
| 1376de4 | Drew DeVault | 27 April 2017, 02:37:15 UTC | Update .build.yml | 27 April 2017, 02:37:15 UTC |
| 11b4c34 | Drew DeVault | 26 April 2017, 21:20:12 UTC | Merge pull request #1196 from SirCmpwn/no_focus Implement no_focus | 26 April 2017, 21:20:12 UTC |
| 537261f | Drew DeVault | 26 April 2017, 21:14:24 UTC | Fix handling of floating windows | 26 April 2017, 21:14:24 UTC |
| ffd0d02 | Drew DeVault | 26 April 2017, 19:35:54 UTC | Update sway(5) | 26 April 2017, 19:35:54 UTC |
| 51143a7 | Drew DeVault | 26 April 2017, 19:29:42 UTC | Implement no_focus Ref #2 | 26 April 2017, 19:29:42 UTC |
| 3c1fc00 | Drew DeVault | 26 April 2017, 14:18:03 UTC | Merge pull request #1194 from snoack/version Add -DVERSION flag for release version numbers | 26 April 2017, 14:18:03 UTC |
| 586eca0 | Drew DeVault | 26 April 2017, 14:07:32 UTC | Fix link to dotfiles, closes #1193 | 26 April 2017, 14:07:32 UTC |
| 173b338 | Sebastian Noack | 26 April 2017, 09:29:30 UTC | Add -DVERSION flag for release version numbers | 26 April 2017, 13:56:46 UTC |
| aca0710 | Drew DeVault | 24 April 2017, 07:08:28 UTC | Merge pull request #1187 from 4e554c4c/master Fix typo in sway(5) | 24 April 2017, 07:08:28 UTC |
| 0dc7e9f | Calvin Lee | 24 April 2017, 03:55:30 UTC | Fix typo in sway(5) | 24 April 2017, 03:57:23 UTC |
| d2de522 | Drew DeVault | 20 April 2017, 17:00:56 UTC | Merge pull request #1184 from JerziKaminsky/logging2 More Logging Fixes | 20 April 2017, 17:00:56 UTC |
| 39a65a6 | Drew DeVault | 20 April 2017, 16:55:34 UTC | Merge branch 'master' into logging2 | 20 April 2017, 16:55:34 UTC |
| 8ecb490 | Jerzi Kaminsky | 20 April 2017, 16:13:53 UTC | Make sway_abort() report location | 20 April 2017, 16:20:40 UTC |
| 382b678 | Drew DeVault | 19 April 2017, 13:17:00 UTC | Merge pull request #1183 from sleep-walker/master explicitly ignore unused return value | 19 April 2017, 13:17:00 UTC |
| 974be01 | Tomáš Čech | 19 April 2017, 08:15:11 UTC | explicitly ignore unused return value fixes issue#1182 | 19 April 2017, 08:16:24 UTC |
| 94ad169 | Drew DeVault | 18 April 2017, 21:57:07 UTC | Merge pull request #1181 from wasamasa/bugfix-initial-scratchpad-size Use i3's dimensions for initial scratchpad views | 18 April 2017, 21:57:07 UTC |
| 35603b2 | Drew DeVault | 18 April 2017, 21:49:23 UTC | Merge pull request #1178 from 4e554c4c/fork Prevent sway from duplicating on a failed fork | 18 April 2017, 21:49:23 UTC |
| 0f31323 | Vasilij Schneidermann | 18 April 2017, 21:47:09 UTC | Use i3's dimensions for initial scratchpad views See https://github.com/i3/i3/blob/master/src/scratchpad.c#L196-L197 | 18 April 2017, 21:47:09 UTC |
| cee2650 | Calvin Lee | 18 April 2017, 03:45:07 UTC | Prevent sway from duplicating on a failed fork Also remove a useless `sway_log` and replace it with a pipe | 18 April 2017, 21:25:37 UTC |
| 17593ac | Drew DeVault | 18 April 2017, 19:50:54 UTC | Merge pull request #1180 from wasamasa/feature-move-to-scratchpad Feature move to scratchpad | 18 April 2017, 19:50:54 UTC |
| d4ac594 | Vasilij Schneidermann | 18 April 2017, 19:35:57 UTC | Implement and document `move [to] scratchpad` | 18 April 2017, 19:41:28 UTC |
| 3c02aa1 | Drew DeVault | 18 April 2017, 17:09:14 UTC | Add .build.yml (for builds.sr.ht) | 18 April 2017, 17:15:23 UTC |
| 7494a48 | Drew DeVault | 16 April 2017, 14:17:43 UTC | Merge pull request #1173 from JerziKaminsky/security_resolve_symlink FOR_REVIEW: IPC security - Allow policy targets to be symlinks | 16 April 2017, 14:17:43 UTC |
| 2ad8850 | Jerzi Kaminsky | 16 April 2017, 06:30:14 UTC | Handle symlinks as IPC security targets - When policies are allocated, the ipc target path goes through symlink resolution. The result is used as the canonical for matching pids to policies at runtime. In particular, this matches up with the target of the `/proc/<pid>/exe`. - There's a possible race condition if this isn't done correctly, read below. Originally, validate_ipc_target() always tried to resolve its argument for symlinks, and returned a parogram target string if it validates. This created a possible race condition with security implications. The problem is that get_feature_policy() first independently resolved the policy target in order to check whether a policy already exists. If it didn't find any, it called alloc_feature_policy() which called validate_ipc_target() which resolved the policy target again. In the time between the two checks, the symlink could be altered, and a lucky attacker could fool the program into thinking that a policy doesn't exist for a target, and then switch the symlink to point at another file. At the very least this could allow him to create two policies for the same program target, and possibly to bypass security by associating the permissions for one target with another, or force default permissions to apply to a target for which a more specific rule has been configured. So we don't that. Instead, the policy target is resolved once and that result is used for the rest of the lookup/creation process. | 16 April 2017, 14:09:53 UTC |
| c9694ee | Jerzi Kaminsky | 14 April 2017, 20:37:43 UTC | Add resolve_path() to utils | 16 April 2017, 14:09:53 UTC |
| bcf9338 | Jerzi Kaminsky | 15 April 2017, 15:54:42 UTC | Add validate_ipc_target() | 16 April 2017, 14:09:53 UTC |
| bfb9923 | Jerzi Kaminsky | 15 April 2017, 14:16:32 UTC | Move get_feature_policy to sway/security.c | 16 April 2017, 14:09:53 UTC |
| b4357a8 | Jerzi Kaminsky | 15 April 2017, 14:04:04 UTC | Rename get_policy to get_feature_policy | 16 April 2017, 14:09:53 UTC |
| cf5e764 | Jerzi Kaminsky | 15 April 2017, 14:13:28 UTC | Disambiguate get_*_policy() and get_*_policy_mask() | 16 April 2017, 14:09:53 UTC |
| edb8075 | Drew DeVault | 16 April 2017, 13:50:50 UTC | Merge pull request #1175 from JerziKaminsky/fix_sway_assert_variadic Fix multiple issues in sway_assert | 16 April 2017, 13:50:50 UTC |
| 709b53b | Jerzi Kaminsky | 16 April 2017, 09:07:43 UTC | Fix location reported by sway_assert | 16 April 2017, 13:39:53 UTC |
| eb3b1ec | Jerzi Kaminsky | 16 April 2017, 07:30:17 UTC | Fix variadic forwarding in sway_assert _sway_assert is a variadic function which tries to delegate to another variadic function. This requires a vprintf-style variant of the delegate. https://stackoverflow.com/a/150616 | 16 April 2017, 13:39:53 UTC |
| 4b3e533 | Drew DeVault | 14 April 2017, 14:44:30 UTC | Merge pull request #1171 from JerziKaminsky/misc_fixes Misc fixes | 14 April 2017, 14:44:30 UTC |
| 6079e6f | Jerzi Kaminsky | 12 April 2017, 17:20:53 UTC | Fix style in sway-bar(5) manpage | 13 April 2017, 23:30:41 UTC |
| b463fb8 | Jerzi Kaminsky | 12 April 2017, 17:13:22 UTC | Remain compat with libinput < 1.7.0 | 13 April 2017, 23:30:41 UTC |
| a25a2f4 | Drew DeVault | 12 April 2017, 14:15:38 UTC | Merge pull request #1147 from SirCmpwn/nvidia-support Downgrade nvidia proprietary driver warning | 12 April 2017, 14:15:38 UTC |
| b3541d9 | Drew DeVault | 12 April 2017, 14:04:28 UTC | Implement more thourough nvidia config check | 12 April 2017, 14:04:44 UTC |
| b036ad9 | Drew DeVault | 03 April 2017, 11:44:36 UTC | Downgrade nvidia proprietary driver warning | 12 April 2017, 14:04:44 UTC |
| cd3c48f | Drew DeVault | 10 April 2017, 12:19:02 UTC | Merge pull request #1146 from SirCmpwn/pretty-print-swaymsg Add pretty printing to swaymsg | 10 April 2017, 12:19:02 UTC |
| 5d3a02a | Drew DeVault | 10 April 2017, 11:17:47 UTC | Merge branch 'master' into pretty-print-swaymsg | 10 April 2017, 11:17:47 UTC |
| 7fee555 | Drew DeVault | 08 April 2017, 10:53:22 UTC | Merge pull request #1150 from JerziKaminsky/cmake_find_libcap Add libcap check to CMake | 08 April 2017, 10:53:22 UTC |
| 8d191b2 | Drew DeVault | 07 April 2017, 18:17:01 UTC | Merge pull request #1155 from 4e554c4c/get_marks Add `-t get_marks` and use more i3-like marks | 07 April 2017, 18:17:01 UTC |
| 154c671 | Calvin Lee | 07 April 2017, 17:37:51 UTC | Add `-t get_marks` and use more i3-like marks In i3 every mark is unique and one mark cannot be used in more than one window, sway behavior has been amended to match this. `swaymsg -t get_marks` will now return an array of all marks used in sway. See #98 | 07 April 2017, 17:37:51 UTC |
| 4e12bf6 | Drew DeVault | 07 April 2017, 12:59:29 UTC | Merge pull request #1153 from SirCmpwn/fix-1152 Fix dangling file descriptors (fixes #1152) | 07 April 2017, 12:59:29 UTC |
| cf95aeb | Drew DeVault | 07 April 2017, 12:49:47 UTC | Merge pull request #1154 from ascent12/fix-1152 Removed explicitly setting file descriptors to -1 | 07 April 2017, 12:49:47 UTC |
| 1b2cd93 | Scott Anderson | 07 April 2017, 12:45:31 UTC | Removed explicitly setting file descriptors to -1 | 07 April 2017, 12:45:31 UTC |
| 50a31a6 | Drew DeVault | 07 April 2017, 11:55:35 UTC | Fix dangling file descriptors (fixes #1152) | 07 April 2017, 11:55:35 UTC |
| aa6bd85 | JerziKaminsky | 05 April 2017, 23:42:25 UTC | Add libcap check to CMake - Moved ``<sys/capability.h>`` include inside `__linux__` guard, because all uses are similarly guarded. - <sys/capability.h> is part of an optional devel package, at least in fedora. CMake now explicitly checks that libcap devel files are available. - Added libcap to the list of install packages in .travis.yml, to make the dependency explicit. travis-ci installs the package by default, which is why this hasn't surfaced previously. | 07 April 2017, 00:14:16 UTC |
| 270e01c | Drew DeVault | 06 April 2017, 12:51:22 UTC | Merge pull request #1151 from ascent12/master Change regex to use PCRE | 06 April 2017, 12:51:22 UTC |
| 3a32be6 | Scott Anderson | 06 April 2017, 12:19:14 UTC | Added designated initaliser, to prevent any possible problem with ordering | 06 April 2017, 12:34:33 UTC |
| fe54a67 | Scott Anderson | 06 April 2017, 12:16:06 UTC | Changed regular expressions to use PCRE for i3 compatibility | 06 April 2017, 12:16:06 UTC |
| 3f40b61 | Drew DeVault | 06 April 2017, 10:47:26 UTC | Merge pull request #1149 from 4e554c4c/criteria Improve criteria handling | 06 April 2017, 10:47:26 UTC |
| 069d37f | Calvin Lee | 05 April 2017, 03:20:27 UTC | Improve criteria handling This commit changes how commands decide what container to act on. Commands get the current container though `current_container`, a global defined in sway/commands.c. If a criteria is given before a command, then the following command will be run once for every container the criteria matches with a reference to the matching container in 'current_container'. Commands should use this instead of `get_focused_container()` from now on. This commit also fixes a few (minor) mistakes made in implementing marks such as non-escaped arrows in sway(5) and calling the "mark" command "floating" by accident. It also cleans up `criteria.c` in a few places. | 06 April 2017, 04:07:23 UTC |
| 7d43a76 | Drew DeVault | 03 April 2017, 18:52:20 UTC | Merge pull request #1145 from 4e554c4c/marks Impliment i3-style marks | 03 April 2017, 18:52:20 UTC |
| 2445d27 | Calvin Lee | 02 April 2017, 20:38:33 UTC | Impliment i3-style marks This commit adds three commands to sway: `show_marks`, `mark` and `unmark`. Marks are displayed right-aligned in the window border as i3 does. Marks may be found using criteria. Fixes #1007 | 03 April 2017, 17:48:37 UTC |
| ab7570d | Drew DeVault | 03 April 2017, 11:46:11 UTC | Update man page | 03 April 2017, 11:46:11 UTC |
| 60ce81e | Drew DeVault | 03 April 2017, 11:27:25 UTC | Add pretty printing to swaymsg If stdout is a tty, it will pretty print unless -r (--raw) is given. Sample outputs: ``` ~/s/s/build > ./bin/swaymsg fullscreen toggle Error: Permission denied for fullscreen toggle via IPC ~/s/s/build > ./bin/swaymsg -t get_workspaces Workspace 3:三 Output: DVI-I-1 Layout: splith Workspace 1:一 (off-screen) Output: HDMI-A-1 Layout: splith Workspace 5:五 (focused) Output: HDMI-A-1 Layout: splith ~/s/s/build > ./bin/swaymsg -t get_inputs Input device Metadot - Das Keyboard Das Keyboard Type: Keyboard Sway ID: 9456:320:Metadot_-_Das_Keyboard_Das_Keyb Input device Wacom Intuos S 2 Pen Type: Tablet tool Sway ID: 1386:827:Wacom_Intuos_S_2 Input device Wacom Intuos S 2 Pad Type: Tablet pad Sway ID: 1386:827:Wacom_Intuos_S_2 Input device Logitech Gaming Mouse G502 Type: Keyboard, Mouse Sway ID: 1133:49277:Logitech_Gaming_Mous ~/s/s/build > ./bin/swaymsg -t get_outputs Output DVI-I-1 Geometry: 1920x1080 @ 3840,0 Scale factor: 1x Workspace: 3:三 Output DVI-D-1 Geometry: 1920x1080 @ 0,0 Scale factor: 1x Workspace: 4:四 Output HDMI-A-1 Geometry: 1920x1080 @ 1920,0 Scale factor: 1x Workspace: 5:五 ``` | 03 April 2017, 11:27:25 UTC |
| 8d9a928 | Drew DeVault | 28 March 2017, 11:45:16 UTC | Update README.md | 28 March 2017, 11:45:16 UTC |
| 82729ee | Drew DeVault | 27 March 2017, 11:02:08 UTC | Mention cap_sys_tty_config in readme | 27 March 2017, 11:02:08 UTC |
| e659db4 | Drew DeVault | 19 March 2017, 21:57:44 UTC | Merge pull request #1130 from oranenj/fix_move_next_crash Fix move next crash | 19 March 2017, 21:57:44 UTC |
| 924ed64 | Jarkko Oranen | 19 March 2017, 14:46:27 UTC | Prevent "move next" and "move prev" commands from crashing Fixes #1120 When the parent of a view is C_WORKSPACE and the movement direction is either MOVE_PREV or MOVE_NEXT, the code would attempt to move the views to the next output, but swayc_adjacent_output can't accept non-directional movement commands and causes undefined behaviour and a segfault. If the code is simply skipped, we end up in an infinite loop. Instead, we can allow containers whose parent is a C_WORKSPACE take the path that handles MOVE_PREV and MOVE_NEXT, which behaves as you would expect. I'm not certain that this fix is entirely correct as the desired behaviour of move_container is not very well defined, but it seems to work. | 19 March 2017, 14:58:39 UTC |
| d735091 | Jarkko Oranen | 19 March 2017, 14:35:04 UTC | The default layout of a workspace should follow the output Hardcoding it to L_HORIZ does not make sense to me, as you get the unexpected behaviour that windows will be arranged horizontally until you switch the layout. | 19 March 2017, 14:57:05 UTC |
