sway

commit 3e61718053a7e2cdd94c64f5613c6bdf28e35c8e
parent 3852a710ca62a76f4fd9c6ff9d28e7e5d5fcf78d
Author: Tony Crisci <tony@dubstepdish.com>
Date:   Wed,  4 Apr 2018 18:52:38 -0400

try to fix focus-inactive

Diffstat:
M
sway/input/seat.c
 | 
20
++++++++------------
M
sway/tree/container.c
 | 
10
++++++++--

2 files changed, 16 insertions(+), 14 deletions(-)
diff --git a/sway/input/seat.c b/sway/input/seat.c
@@ -467,22 +467,18 @@ struct sway_container *sway_seat_get_focus(struct sway_seat *seat) {
 
 struct sway_container *seat_get_focus_by_type(struct sway_seat *seat,
 		struct sway_container *container, enum sway_container_type type) {
+	if (container->type == C_VIEW || container->children->length == 0) {
+		return container;
+	}
+
 	struct sway_seat_container *current = NULL;
-	struct sway_container *parent = NULL;
 	wl_list_for_each(current, &seat->focus_stack, link) {
-		parent = current->container->parent;
-
-		if (current->container == container &&
-				(type == C_TYPES || container->type == type)) {
-			return current->container;
+		if (type != C_TYPES && container->type != type) {
+			continue;
 		}
 
-		while (parent) {
-			if (parent == container && (type == C_TYPES ||
-						current->container->type == type)) {
-				return current->container;
-			}
-			parent = parent->parent;
+		if (container_has_child(container, current->container)) {
+			return current->container;
 		}
 	}
 
diff --git a/sway/tree/container.c b/sway/tree/container.c
@@ -51,12 +51,18 @@ const char *container_type_to_str(enum sway_container_type type) {
 }
 
 void container_create_notify(struct sway_container *container) {
+	if (container->type != C_VIEW || container->type != C_CONTAINER) {
+		return;
+	}
 	// TODO send ipc event type based on the container type
 	wl_signal_emit(&root_container.sway_root->events.new_container, container);
 	ipc_event_window(container, "new");
 }
 
 static void container_close_notify(struct sway_container *container) {
+	if (container == NULL) {
+		return;
+	}
 	// TODO send ipc event type based on the container type
 	ipc_event_window(container, "close");
 }
@@ -522,8 +528,8 @@ static bool find_child_func(struct sway_container *con, void *data) {
 
 bool container_has_child(struct sway_container *con,
 		struct sway_container *child) {
-	if (child == NULL || child->type == C_VIEW ||
-			child->children->length == 0) {
+	if (con == NULL || con->type == C_VIEW ||
+			con->children->length == 0) {
 		return false;
 	}
 	return container_find(con, find_child_func, child);