Prevent passing WM keys, improve multihead support

sway/handlers.c

@@ -57,7 +57,6 @@ bool handle_key(wlc_handle view, uint32_t time, const struct wlc_modifiers
 	// Lowercase if necessary
 	sym = tolower(sym);
 
-	if (state == WLC_KEY_STATE_PRESSED) {
 	int i;
 	for (i = 0; i < mode->bindings->length; ++i) {
 		struct sway_binding *binding = mode->bindings->items[i];
@@ -74,7 +73,11 @@ bool handle_key(wlc_handle view, uint32_t time, const struct wlc_modifiers
 			}
 
 			if (match) {
-					cmd_success = handle_command(config, binding->command);
+				// TODO: --released
+				if (state == WLC_KEY_STATE_PRESSED) {
+					cmd_success = !handle_command(config, binding->command);
+				} else {
+					cmd_success = true;
 				}
 			}
 		}

sway/layout.c

@@ -4,6 +4,7 @@
 #include "list.h"
 #include "log.h"
 #include "layout.h"
+#include "container.h"
 #include "workspace.h"
 
 swayc_t root_container;
@@ -35,10 +36,16 @@ void arrange_windows(swayc_t *container, int width, int height) {
 		height = container->height;
 	}
 
+	int x = 0, y = 0;
 	switch (container->type) {
 	case C_ROOT:
 		for (i = 0; i < container->children->length; ++i) {
-			arrange_windows(container->children->items[i], -1, -1);
+			swayc_t *child = container->children->items[i];
+			sway_log(L_DEBUG, "Arranging output at %d", x);
+			child->x = x;
+			child->y = y;
+			arrange_windows(child, child->width, child->height);
+			x += child->width;
 		}
 		return;
 	case C_VIEW:
@@ -85,7 +92,6 @@ void arrange_windows(swayc_t *container, int width, int height) {
 		total_weight += child->weight;
 	}
 
-	int x = 0, y = 0;
 	switch (container->layout) {
 	case L_HORIZ:
 	default:
@@ -307,6 +313,13 @@ swayc_t *create_container(swayc_t *parent, wlc_handle handle) {
 	return c;
 }
 
+void add_output_widths(swayc_t *container, void *_width) {
+	int *width = _width;
+	if (container->type == C_OUTPUT) {
+		*width += container->width;
+	}
+}
+
 void add_output(wlc_handle output) {
 	sway_log(L_DEBUG, "Adding output %d", output);
 	const struct wlc_size* size = wlc_output_get_resolution(output);
@@ -317,6 +330,9 @@ void add_output(wlc_handle output) {
 	container->height = size->h;
 	add_child(&root_container, container);
 
+	int total_width = 0;
+	container_map(&root_container, add_output_widths, &total_width);
+
 	swayc_t *workspace = create_container(container, -1);
 	workspace->type = C_WORKSPACE;
 	workspace->name = workspace_next_name();
@@ -324,6 +340,7 @@ void add_output(wlc_handle output) {
 	workspace->height = size->h;
 	workspace->layout = L_HORIZ; // TODO: Get default layout from config
 	add_child(container, workspace);
+	sway_log(L_DEBUG, "Added workspace %s for output %d", workspace->name, output);
 
 	workspace_switch(workspace);
 

sway/workspace.c

@@ -9,10 +9,18 @@
 
 swayc_t *active_workspace = NULL;
 
+int ws_num = 1;
+
 char *workspace_next_name(void) {
-	//TODO change this i guess. seems pretty bad
+	int l = 1;
-	char *name = malloc(sizeof("1"));
+	if (ws_num >= 10) {
-	return strcpy(name, "1");
+		l = 2;
+	} else if (ws_num >= 100) {
+		l = 3;
+	}
+	char *name = malloc(l + 1);
+	sprintf(name, "%d", ws_num++);
+	return name;
 }
 
 swayc_t *workspace_create(const char* name) {
@@ -45,7 +53,6 @@ bool workspace_destroy(swayc_t *workspace) {
 	sway_log(L_DEBUG, "Workspace: Destroying workspace '%s'", workspace->name);
 	free_swayc(workspace);
 	return true;
-
 }
 
 void set_mask(swayc_t *view, void *data) {